Skip to content
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 pre-commit hooks #34

Closed
wants to merge 2 commits into from

Conversation

Copy link

codecov bot commented Mar 12, 2024

Codecov Report

All modified and coverable lines are covered by tests βœ…

Project coverage is 18.4%. Comparing base (26bbfae) to head (d5ea2d8).

❗ Current head d5ea2d8 differs from pull request most recent head 7fcb660

Please upload reports for the commit 7fcb660 to get more accurate results.

Additional details and impacted files
@@           Coverage Diff           @@
##            main     #34     +/-   ##
=======================================
- Coverage   18.4%   18.4%   -0.1%     
=======================================
  Files         99      99             
  Lines      16599   16599             
  Branches    2808    2808             
=======================================
- Hits        3059    3057      -2     
- Misses     13540   13542      +2     

β˜” View full report in Codecov by Sentry.
πŸ“’ Have feedback on the report? Share it here.

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch 2 times, most recently from 8e90b0f to cc45a49 Compare March 25, 2024 17:34
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from adb452e to ffbe8fe Compare April 1, 2024 17:34
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch 2 times, most recently from 3dbb8af to 1255246 Compare April 15, 2024 17:39
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 282d8b3 to 44e828e Compare April 22, 2024 17:33
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch 2 times, most recently from efe0af8 to dfbe37c Compare May 6, 2024 17:37
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 39eb0e2 to aedfc6f Compare May 13, 2024 17:30
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 32ff116 to cca9e96 Compare May 27, 2024 17:32
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 565436a to 6452902 Compare June 3, 2024 17:35
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 600c6dc to fcdf01a Compare June 10, 2024 17:38
updates:
- [github.com/pre-commit/pre-commit-hooks: v4.5.0 β†’ v4.6.0](pre-commit/pre-commit-hooks@v4.5.0...v4.6.0)
- [github.com/srstevenson/nb-clean: 3.2.0 β†’ 3.3.0](srstevenson/nb-clean@3.2.0...3.3.0)
- [github.com/sirosen/texthooks: 0.6.4 β†’ 0.6.6](sirosen/texthooks@0.6.4...0.6.6)
- [github.com/pre-commit/mirrors-clang-format: v17.0.6 β†’ v18.1.6](pre-commit/mirrors-clang-format@v17.0.6...v18.1.6)
- [github.com/astral-sh/ruff-pre-commit: v0.2.2 β†’ v0.4.9](astral-sh/ruff-pre-commit@v0.2.2...v0.4.9)
- [github.com/codespell-project/codespell: v2.2.6 β†’ v2.3.0](codespell-project/codespell@v2.2.6...v2.3.0)
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 0451b5b to 77bd177 Compare June 17, 2024 17:35
@burgholzer
Copy link
Member

Superseded by #35

@burgholzer burgholzer closed this Jun 17, 2024
Copy link

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-tidy reports: 2 concern(s)
  • extern/qfr/extern/dd_package/include/dd/QuantumCircuitSimulation.hpp:249:14: warning: [clang-diagnostic-shadow]

    declaration shadows a local variable

            auto qmddResult  = qmdd->determineMeasurementProbabilities(qmddState, i, true);
                 ^
    /home/runner/work/mqt-limdd/mqt-limdd/extern/qfr/extern/dd_package/include/dd/QuantumCircuitSimulation.hpp:244:21: note: previous declaration is here
        auto            qmddResult  = qmdd->measureAll(qmddState, false, mt);
                        ^
  • extern/qfr/extern/dd_package/include/dd/QuantumCircuitSimulation.hpp:250:14: warning: [clang-diagnostic-shadow]

    declaration shadows a local variable

            auto limddResult = limdd->determineMeasurementProbabilities(limddState, i, true);
                 ^
    /home/runner/work/mqt-limdd/mqt-limdd/extern/qfr/extern/dd_package/include/dd/QuantumCircuitSimulation.hpp:245:21: note: previous declaration is here
        auto            limddResult = qmdd->measureAll(limddState, false, mt);
                        ^

Have any feedback or feature suggestions? Share it here.

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.

1 participant