Skip to content

Apply SAFE ruff check auto-fixes - #262

Draft
Padraic Shafer (padraic-shafer) wants to merge 3 commits into
NSLS2:mainfrom
padraic-shafer:ruff-fix-2026-3
Draft

Apply SAFE ruff check auto-fixes#262
Padraic Shafer (padraic-shafer) wants to merge 3 commits into
NSLS2:mainfrom
padraic-shafer:ruff-fix-2026-3

Conversation

@padraic-shafer

Copy link
Copy Markdown
Collaborator

Linter updates automatically applied by running:

uvx ruff check --fix

Changes include:

  • Removed unused imports.
  • Import order enforced.
  • Prefers | over older Union[].
  • Prefers list, tuple over older List, Tuple, etc.
  • Prefers f-string conversion flags, such as !s over str()
  • Removes explicit pass if not required [I manually added a comment in its place]
  • Logical idiom: min operator
  • Wanted to remove #noqa: E712 [I kept it and added a reason why]

This PR stacks onto #261.

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.

1 participant