PennyLaneAI / pennylane-lightning

The PennyLane-Lightning plugin provides a fast state-vector simulator written in C++ for use with PennyLane
https://docs.pennylane.ai/projects/lightning
Apache License 2.0
92 stars 40 forks source link

Prevent labels from triggering CI #899

Closed anthayes92 closed 2 months ago

anthayes92 commented 2 months ago

Context: CI is being triggered by adding any label to a PR in "ready for review" state. This fixes that undesired behaviour

Description of the Change: Removes the labeled PR type from workflows.

Benefits: Prevents unnecessary resource usage.

Possible Drawbacks: When labels are added that intend to affect CI, a followup commit must be made to trigger the CI

Related GitHub Issues:

Verification:

Added various labels to this PR in a "ready for review" state, no CI checks were triggered as a result.

github-actions[bot] commented 2 months ago

Hello. You may have forgotten to update the changelog! Please edit .github/CHANGELOG.md with:

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.88%. Comparing base (43374cc) to head (dcf2721). Report is 1 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #899 +/- ## ========================================== + Coverage 96.22% 97.88% +1.66% ========================================== Files 212 212 Lines 28187 28187 ========================================== + Hits 27123 27591 +468 + Misses 1064 596 -468 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

AmintorDusko commented 2 months ago

Hey @anthayes92, I just merged master to your PR.

anthayes92 commented 2 months ago

but don't forget to update the CHANGELOG.

Appreciate the reminder, though we typically haven't been adding these GH workflow updates to the changelog since the application code is unchanged