Removed not-context-manager and ordered lint rules

Qiskit / qiskit

Qiskit is an open-source SDK for working with quantum computers at the level of extended quantum circuits, operators, and primitives.

https://www.ibm.com/quantum/qiskit

Apache License 2.0

4.83k stars 2.29k forks source link

Removed not-context-manager and ordered lint rules #12607

Open joesho112358 opened 2 weeks ago

joesho112358 commented 2 weeks ago

Summary

This should close out #9614

Details and comments

Removes not-context-manager and orders lint rules alphabetically

qiskit-bot commented 2 weeks ago

Thank you for opening a new pull request.

Before your PR can be merged it will first need to pass continuous integration tests and be reviewed. Sometimes the review process can be slow, so please be patient.

While you're waiting, please feel free to review other open PRs. While only a subset of people are authorized to approve pull requests for merging, everyone is encouraged to review open pull requests. Doing reviews helps reduce the burden on the core team and helps make the project's code better for everyone.

One or more of the following people are relevant to this code:

@Qiskit/terra-core

coveralls commented 2 weeks ago

Pull Request Test Coverage Report for Build 9586581711

Details

0 of 0 changed or added relevant lines in 0 files are covered.
12 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.01%) to 89.746%

Files with Coverage Reduction	New Missed Lines	%
crates/qasm2/src/expr.rs	1	94.02%
crates/qasm2/src/lex.rs	5	92.62%
crates/qasm2/src/parse.rs	6	96.69%
<!--	Total:	12	-->

Totals
Change from base Build 9585515412:	-0.01%
Covered Lines:	63560
Relevant Lines:	70822

💛 - Coveralls

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9607055660

Details

0 of 0 changed or added relevant lines in 0 files are covered.
12 unchanged lines in 2 files lost coverage.
Overall coverage remained the same at 89.74%

Files with Coverage Reduction	New Missed Lines	%
crates/qasm2/src/lex.rs	6	92.37%
crates/qasm2/src/parse.rs	6	97.61%
<!--	Total:	12	-->

Totals
Change from base Build 9602245093:	0.0%
Covered Lines:	63573
Relevant Lines:	70841

💛 - Coveralls

coveralls commented 5 days ago

Pull Request Test Coverage Report for Build 9706332095

Details

0 of 0 changed or added relevant lines in 0 files are covered.
12 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.006%) to 89.746%

Files with Coverage Reduction	New Missed Lines	%
crates/qasm2/src/lex.rs	6	92.11%
crates/qasm2/src/parse.rs	6	97.15%
<!--	Total:	12	-->

Totals
Change from base Build 9703107599:	0.006%
Covered Lines:	63802
Relevant Lines:	71092