-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
📝 Add docstring check to ruff and fix respective errors #331
Conversation
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.
Nice! Many thanks! 🙏
Just a couple of suggestions where i think it might make sense to combine comments.
src/mqt/bench/benchmarks/qiskit_application_nature/groundstate.py
Outdated
Show resolved
Hide resolved
src/mqt/bench/benchmarks/qiskit_application_finance/portfolioqaoa.py
Outdated
Show resolved
Hide resolved
src/mqt/bench/benchmarks/qiskit_application_finance/portfoliovqe.py
Outdated
Show resolved
Hide resolved
src/mqt/bench/benchmarks/qiskit_application_optimization/routing.py
Outdated
Show resolved
Hide resolved
src/mqt/bench/benchmarks/qiskit_application_optimization/tsp.py
Outdated
Show resolved
Hide resolved
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #331 +/- ##
=======================================
+ Coverage 92.9% 93.1% +0.2%
=======================================
Files 47 47
Lines 2437 2437
=======================================
+ Hits 2264 2270 +6
+ Misses 173 167 -6
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
No description provided.