-
Notifications
You must be signed in to change notification settings - Fork 544
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
Pylinting more tests #3620
Pylinting more tests #3620
Conversation
Hello. You may have forgotten to update the changelog!
|
Codecov Report
@@ Coverage Diff @@
## master #3620 +/- ##
=======================================
Coverage 99.86% 99.86%
=======================================
Files 314 314
Lines 28084 28084
=======================================
Hits 28046 28046
Misses 38 38 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks great! can you fix up the whitespace in the Makefile? I know it's actual tabs.. but git makes it look especially funny
* skip test failing due to networkx version * Update doc/releases/changelog-dev.md * dont run qcut tests in interface runners * fix qcut return types fixture, only selectively skip * return types dont run qcut tests
Bumps [torch](https://github.com/pytorch/pytorch) from 1.11.0+cpu to 1.13.1. - [Release notes](https://github.com/pytorch/pytorch/releases) - [Changelog](https://github.com/pytorch/pytorch/blob/master/RELEASE.md) - [Commits](https://github.com/pytorch/pytorch/commits/v1.13.1) --- updated-dependencies: - dependency-name: torch dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…3624) * avoid using global random numbers generation in qchem * use local random number generation * Update doc/releases/changelog-dev.md * fix is_independent tests * remove spsa change, update random test * undo spsa test change
* add pre-commit hook to run pylint on tests passing pylint * skip test pre-commit if no tests are changed
…nnylane into pylint-tests-more
…into pylint-tests-more
too-many-arguments, | ||
too-few-public-methods, | ||
too-many-public-methods, | ||
no-self-argument, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
where was this raised? This is a funny one to hit
Context:
Adding to the work done here to start linting more of the test files: #3600
Description of the Change:
make lint-test file=./tests/math/test_density_matrices.py
Benefits:
Moar cleaner code.
Possible Drawbacks:
Disabling certain pylint rules might hide issues and doesn't actually improve the code itself. The goal with this PR is not to be super intrusive.
Related GitHub Issues: