Skip to content

Remove unused ruff ignore#195

Merged
AdrianDAlessandro merged 1 commit intomainfrom
ruff-checks
Mar 28, 2025
Merged

Remove unused ruff ignore#195
AdrianDAlessandro merged 1 commit intomainfrom
ruff-checks

Conversation

@AdrianDAlessandro
Copy link
Copy Markdown
Collaborator

Description

I noticed that ignoring this check is no longer needed. So I thought I'd remove the ignore.

Type of change

  • Documentation (non-breaking change that adds or improves the documentation)
  • New feature (non-breaking change which adds functionality)
  • Optimization (non-breaking, back-end change that speeds up the code)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (whatever its nature)

Key checklist

  • All tests pass (eg. pytest)
  • The documentation builds and looks OK (eg. mkdocs)
  • Pre-commit hooks run successfully (eg. pre-commit run --all-files)

Further checks

  • Code is commented, particularly in hard-to-understand areas
  • Tests added or an issue has been opened to tackle that in the future. (Indicate issue here: # (issue))

Copy link
Copy Markdown
Collaborator

@alexdewar alexdewar left a comment

Choose a reason for hiding this comment

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

LGTM!

@AdrianDAlessandro AdrianDAlessandro merged commit 829f838 into main Mar 28, 2025
14 checks passed
@AdrianDAlessandro AdrianDAlessandro deleted the ruff-checks branch March 28, 2025 10:51
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