Closed maliasadi closed 2 months ago
Hello. You may have forgotten to update the changelog! Please edit .github/CHANGELOG.md with:
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 95.79%. Comparing base (
70dfdf7
) to head (fe9968c
). Report is 1 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Before submitting
Please complete the following checklist when submitting a PR:
[ ] All new features must include a unit test. If you've fixed a bug or added code that should be tested, add a test to the
tests
directory![ ] All new functions and code must be clearly commented and documented. If you do make documentation changes, make sure that the docs build and render correctly by running
make docs
.[ ] Ensure that the test suite passes, by running
make test
.[ ] Add a new entry to the
.github/CHANGELOG.md
file, summarizing the change, and including a link back to the PR.[ ] Ensure that code is properly formatted by running
make format
.When all the above are checked, delete everything above the dashed line and fill in the pull request template.
Context: As a followup to #887, this PR fixes the issue with the name of wheels artifacts as per https://github.com/actions/upload-artifact/issues/478
Description of the Change:
Benefits:
Possible Drawbacks:
Related GitHub Issues: [sc-73124]