Closed vincentmr 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 56.95%. Comparing base (
2189af1
) to head (46be977
). Report is 5 commits behind head on master.:exclamation: There is a different number of reports uploaded between BASE (2189af1) and HEAD (46be977). Click for more details.
HEAD has 78 uploads less than BASE
| Flag | BASE (2189af1) | HEAD (46be977) | |------|------|------| ||83|5|
: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:
Description of the Change:
Benefits:
Possible Drawbacks:
Related GitHub Issues: