Sanity check PR - Githubissues

wilsonbb commented 4 months ago

Change Description

[ ] My PR includes a link to the issue that I am addressing

Solution Description

Code Quality

[ ] My code builds (or compiles) cleanly without any errors or warnings
[ ] My code contains relevant comments and necessary documentation

Project-Specific Pull Request Checklists

[ ] I have added a function that requires a sync_tables command, and have added the neccesary sync_tables call

Bug Fix Checklist

[ ] My fix includes a new test that breaks as a result of the bug (if possible)
[ ] My change includes a breaking change
- [ ] My change includes backwards compatibility and deprecation warnings (if possible)

New Feature Checklist

[ ] I have added or updated the docstrings associated with my feature using the NumPy docstring format
[ ] I have updated the tutorial to highlight my new feature (if appropriate)
[ ] I have added unit/End-to-End (E2E) test cases to cover my new feature
[ ] My change includes a breaking change
- [ ] My change includes backwards compatibility and deprecation warnings (if possible)

Documentation Change Checklist

[ ] Any updated docstrings use the NumPy docstring format

Build/CI Change Checklist

[ ] If required or optional dependencies have changed (including version numbers), I have updated the README to reflect this
[ ] If this is a new CI setup, I have added the associated badge to the README

Other Change Checklist

[ ] Any new or updated docstrings use the NumPy docstring format.
[ ] I have updated the tutorial to highlight my new feature (if appropriate)
[ ] I have added unit/End-to-End (E2E) test cases to cover any changes
[ ] My change includes a breaking change
- [ ] My change includes backwards compatibility and deprecation warnings (if possible)

github-actions[bot] commented 4 months ago

Before [89aa1169]	After [a0909acd]	Ratio	Benchmark (Parameter)
36.0±0.4ms	36.4±0.4ms	1.01	benchmarks.time_prune_sync_workflow
35.4±0.4ms	35.2±0.2ms	1	benchmarks.time_batch

Click here to view all benchmarks.

codecov[bot] commented 4 months ago

Codecov Report

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

Project coverage is 95.50%. Comparing base (89aa116) to head (9df5102).

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #430 +/- ## ========================================== - Coverage 95.84% 95.50% -0.34% ========================================== Files 25 25 Lines 1781 1781 ========================================== - Hits 1707 1701 -6 - Misses 74 80 +6 ```

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