Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR is intended to be merged after #386 .
It enables the bandit set of linting checks for the repository along with making some of the needed fixes. As my main priority with these changes is to ensure the full set of planned linting checks are in place for new development efforts, some path and line-based ignores have been applied.
In addition, this PR also in a slight decrease in test coverage. However, because none of the replaced
assert
statements were triggered in previous test cases this is a coverage decrease in form but not function. As such, I propose merging this PR with the improvements already present, and tackle the missing coverage during the planned post-core refactor.Note: For review purposes this is currently targeting the branch cytomining:ci/update-ruff-to-0.3.4. It will be switched to target main after that branch is merged.
What is the nature of your change?
Checklist
Please ensure that all boxes are checked before indicating that a pull request is ready for review.