Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DM-48788: Switch to ruff format #106

Merged
merged 7 commits into from
Feb 7, 2025
Merged

DM-48788: Switch to ruff format #106

merged 7 commits into from
Feb 7, 2025

Conversation

timj
Copy link
Member

@timj timj commented Feb 5, 2025

Checklist

  • ran Jenkins
  • added a release note for user-visible changes to doc/changes

Copy link

codecov bot commented Feb 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.80%. Comparing base (c989d23) to head (f0cfc68).
Report is 8 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #106   +/-   ##
=======================================
  Coverage   86.80%   86.80%           
=======================================
  Files          27       27           
  Lines        4690     4690           
  Branches      566      566           
=======================================
  Hits         4071     4071           
  Misses        474      474           
  Partials      145      145           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@timj timj force-pushed the tickets/DM-48788 branch from af6cea5 to a7fbf13 Compare February 6, 2025 21:49
@timj timj force-pushed the tickets/DM-48788 branch from f013b33 to 10cf241 Compare February 6, 2025 21:54
@timj timj requested a review from dhirving February 6, 2025 22:07
@timj
Copy link
Member Author

timj commented Feb 6, 2025

@dhirving please take a quick look. I'm going to start going through the packages but I won't be asking for reviews each time because I want to minimize the time that the PRs are open and they are all going to be mechanical automated changes. This is representative though.

Copy link
Contributor

@dhirving dhirving left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. It's exciting that it auto-sorts the __all__ lists and fixes docstring line lengths.

@timj timj merged commit 2cf0f8f into main Feb 7, 2025
21 checks passed
@timj timj deleted the tickets/DM-48788 branch February 7, 2025 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants