FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

STY: pin ruff version exactly and fix new violations in v0.16.0 by neutrinoceros · Pull Request #411 · OpenAstronomy/github-actions-workflows · GitHub

STY: pin ruff version exactly and fix new violations in v0.16.0 - #411

Merged
zacharyburnett merged 1 commit into
OpenAstronomy:mainfrom
neutrinoceros:sty/pin-ruff
Aug 11, 2026
Merged

STY: pin ruff version exactly and fix new violations in v0.16.0#411
zacharyburnett merged 1 commit into
OpenAstronomy:mainfrom
neutrinoceros:sty/pin-ruff

Conversation

Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor Author

errors are unrelated and addressed in #410.
Note that both PRs include updates to helper scripts (and consequently to their hashes), which will necessarily lead to some merge conflicts. I advise merging one, then rebase/update the other one.

zacharyburnett commented Jul 28, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

we can also explicitly select the old default ruff rules and add in the new ones one at a time, like in spacetelescope/romancal#2400; the docs say the pre-0.16.0 default rules were:

select = ["E4", "E7", "E9", "F"]

Copy link
Copy Markdown
Contributor Author

That'd be fine with me, just not guaranteed to work through future (implicit) upgrades. Let me know what you prefer. In the mean time, I'll just resolve merge conflicts.

zacharyburnett left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

ok then, lets go with the new default rules that ship with 0.16.0, they seem sensible enough and don't introduce too many changes

zacharyburnett merged commit 2ee1c83 into OpenAstronomy:main Aug 11, 2026
121 of 122 checks passed
neutrinoceros deleted the sty/pin-ruff branch August 11, 2026 14:23
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL