Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@NickleDave
Copy link
Collaborator

@NickleDave NickleDave commented Aug 14, 2023

This adds linting dependencies (isort, black, flake8) and a linting session to the noxfile that runs those.

I also ran them all to make an initial pass at cleaning up. Will still need to fix one or two stray warnings.

- remove dict type hinting that I think was not valid anyways
- use isinstance instead of type == numpy.ndarray
- fix an incorrect variable name in an error message
- Use isinstance checks instead of type ==
- Add missing values in error message (printed with f-strings
  to help user find root of issue)
… triggering invalid escape sequence warning from flake8
@NickleDave NickleDave merged commit a96ff97 into main Aug 14, 2023
@NickleDave NickleDave deleted the fix-and-run-linting branch August 28, 2023 16:42
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