-
Notifications
You must be signed in to change notification settings - Fork 61
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
Update pylint requirement from 3.1.1 to 3.2.0 #584
Conversation
Updates the requirements on [pylint](https://github.com/pylint-dev/pylint) to permit the latest version. - [Release notes](https://github.com/pylint-dev/pylint/releases) - [Commits](pylint-dev/pylint@v3.1.1...v3.2.0) --- updated-dependencies: - dependency-name: pylint dependency-type: direct:development ... Signed-off-by: dependabot[bot] <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #584 +/- ##
======================================
Coverage 98.1% 98.1%
======================================
Files 162 162
Lines 3108 3108
Branches 760 760
======================================
Hits 3050 3050
Misses 37 37
Partials 21 21 ☔ View full report in Codecov by Sentry. |
@@ -116,7 +116,7 @@ def test_partial_channel(test_input, expected, sys_arg, dim_arg): | |||
elif sys_arg is not None and dim_arg is not None: | |||
calculated = partial_channel(test_input, depolarizing(2), sys_arg, dim_arg) | |||
|
|||
assert np.isclose(calculated, expected).all() | |||
assert np.isclose(calculated, expected).all() # pylint: disable=possibly-used-before-assignment |
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.
Made pylint ignore this line because it doesn't see the conditional assignment in the previous lines.
@@ -91,7 +91,7 @@ def is_unextendible_product_basis(vecs: list[np.ndarray], dims: list[int]) -> tu | |||
# Acquire generator to m-partitions of [0, n-1]. | |||
parts_unordered = set_partitions(list(range(num_vecs)), num_parties) | |||
|
|||
for part_unordered in parts_unordered: | |||
for part_unordered in parts_unordered: # pylint: disable=not-an-iterable |
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.
added this line because I think it is a false positive. If this gets fixed in a newer release, we should get a warning about useless suppression
.
Updates the requirements on pylint to permit the latest version.
Commits
b738d23
Bump pylint to 3.2.0, update changelog040ce17
Upgrade the contributors lists743a04d
[performance] Check that 'trailing-comma-tuple' is enabled only once (#9620)b4a9535
[pre-commit.ci] pre-commit autoupdate (#9621)a4f9b86
[pre-commit.ci] pre-commit autoupdate4f0c7ac
Merge maintenance/3.1.x into main following 3.1.1 release (#9618)0d8ef10
Merge maintenance/3.1.x into main following 3.1.1 releasee53fdb6
Bump furo from 2024.4.27 to 2024.5.6 (#9617)814bf94
Bump actions/checkout from 4.1.4 to 4.1.5 (#9616)fd6790b
Add ignored modules to Astroid module deny list (#9504)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase
.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebase
will rebase this PR@dependabot recreate
will recreate this PR, overwriting any edits that have been made to it@dependabot merge
will merge this PR after your CI passes on it@dependabot squash and merge
will squash and merge this PR after your CI passes on it@dependabot cancel merge
will cancel a previously requested merge and block automerging@dependabot reopen
will reopen this PR if it is closed@dependabot close
will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditions
will show all of the ignore conditions of the specified dependency@dependabot ignore this major version
will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor version
will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependency
will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)