Skip to content

Ruff and pyright ig #41

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

Merged
merged 5 commits into from
May 21, 2025
Merged

Ruff and pyright ig #41

merged 5 commits into from
May 21, 2025

Conversation

iangillingham-stfc
Copy link
Contributor

Description of work

Ruff & pyright day: (ISISComputingGroup/IBEX#8553 (comment))
Added loads of type hints.
Removed future imports.
Changed string.format() to f strings for improved clarity.
Some type casting where necessary.

To test

Ruff and pyright

Acceptance criteria

No errors emitted


Code Review

  • Is the code of an acceptable quality?

Functional Tests

  • Do changes function as described? Add comments below that describe the tests performed.

@Tom-Willemsen Tom-Willemsen merged commit 01606f6 into main May 21, 2025
15 checks passed
@Tom-Willemsen Tom-Willemsen deleted the ruff_and_pyright_ig branch May 21, 2025 15:57
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