Skip to content

Add ruff formatter #17

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.

Sign up for GitHub

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

Jump to bottom
Merged
reweeden merged 2 commits into from
Dec 22, 2025
Merged

Add ruff formatter #17

reweeden merged 2 commits into from
Dec 22, 2025

Conversation

@reweeden
Copy link
Contributor

@reweeden reweeden commented Oct 3, 2025 •
edited
Loading

Pull Request Checklist

I have:

  • performed a self review of my code I&A code style
    • Resources and Data Structures are sorted by ABC or a defined sorting pattern
  • updated the documentation accordingly
  • verified required action checks are passing
  • bumped the version number as appropriate

@reweeden reweeden force-pushed the rew/add-ruff-formatter branch from dc4ede7 to f334ff2 Compare December 22, 2025 18:21
@reweeden
Add ruff format check
0398a7e
@reweeden
Remove flake8-commas lint
74351b3 
This is because in preview versions of the ruff formatter, some nested 
brackets will be collapsed so long as there is no trailing comma. In 
these cases we can leave the commas off for now, and when (if) the 
collapsing feature is stabalized, the code will be reformatted 
automatically.
@reweeden reweeden force-pushed the rew/add-ruff-formatter branch from f334ff2 to 74351b3 Compare December 22, 2025 18:27
@reweeden reweeden marked this pull request as ready for review December 22, 2025 18:32
@reweeden reweeden merged commit ac60820 into main Dec 22, 2025
11 checks passed
@reweeden reweeden deleted the rew/add-ruff-formatter branch December 22, 2025 23:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Cameronsplaze Cameronsplaze Cameronsplaze approved these changes

@gjclark gjclark gjclark approved these changes

@benbart benbart Awaiting requested review from benbart

@mattp0 mattp0 Awaiting requested review from mattp0

@mckadesorensen mckadesorensen Awaiting requested review from mckadesorensen

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@reweeden @Cameronsplaze @gjclark