Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

♻️ refactor(quantity): ensure abstract quantities are abstract #348

Merged
merged 1 commit into from
Dec 22, 2024

Conversation

nstarman
Copy link
Contributor

No change to API, just ensuring the abstract classes are abstract.

@nstarman nstarman added the backport v1.0.x on-merge: backport to versions/v1.0.x label Dec 22, 2024
@nstarman nstarman added this to the v1.0.x milestone Dec 22, 2024
Copy link

codecov bot commented Dec 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.76%. Comparing base (acfb136) to head (dfb695e).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #348      +/-   ##
==========================================
+ Coverage   97.74%   97.76%   +0.02%     
==========================================
  Files          44       44              
  Lines        1595     1611      +16     
  Branches      332      332              
==========================================
+ Hits         1559     1575      +16     
  Misses         29       29              
  Partials        7        7              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@nstarman nstarman marked this pull request as ready for review December 22, 2024 21:40
@nstarman nstarman merged commit 5bb134d into GalacticDynamics:main Dec 22, 2024
23 checks passed
@nstarman nstarman deleted the quantity/abstract branch December 22, 2024 21:40
meeseeksmachine pushed a commit to meeseeksmachine/unxt that referenced this pull request Dec 22, 2024
nstarman added a commit that referenced this pull request Dec 22, 2024
…re abstract (#349)

Co-authored-by: Nathaniel Starkman <nstarman@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
backport v1.0.x on-merge: backport to versions/v1.0.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant