Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Add novas_planet_for_name() #87

Merged
merged 1 commit into from
Nov 10, 2024
Merged

Add novas_planet_for_name() #87

merged 1 commit into from
Nov 10, 2024

Conversation

attipaci
Copy link
Collaborator

@attipaci attipaci commented Nov 10, 2024

Also:

  • Additional argument checks for novas_calceph(), and related tests.

@attipaci attipaci added the enhancement New feature or request label Nov 10, 2024
@attipaci attipaci added this to the 1.2.0 milestone Nov 10, 2024
@attipaci attipaci self-assigned this Nov 10, 2024
Copy link

codecov bot commented Nov 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.65%. Comparing base (5082dca) to head (f5170c4).
Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #87   +/-   ##
=======================================
  Coverage   99.65%   99.65%           
=======================================
  Files           8        8           
  Lines        2888     2904   +16     
  Branches      554      561    +7     
=======================================
+ Hits         2878     2894   +16     
  Partials       10       10           
Flag Coverage Δ
unittests 99.65% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
src/super.c 100.00% <100.00%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5aab1e8...f5170c4. Read the comment docs.

@attipaci attipaci force-pushed the planet-by-name branch 2 times, most recently from 298cd99 to 4e9295e Compare November 10, 2024 12:46
@attipaci attipaci marked this pull request as ready for review November 10, 2024 12:47
@attipaci attipaci force-pushed the planet-by-name branch 2 times, most recently from 50836dc to 06080ca Compare November 10, 2024 12:50
@attipaci attipaci merged commit 6a5ae23 into main Nov 10, 2024
16 checks passed
@attipaci attipaci deleted the planet-by-name branch November 10, 2024 13:01
attipaci added a commit that referenced this pull request Nov 10, 2024
attipaci added a commit that referenced this pull request Nov 10, 2024
attipaci added a commit that referenced this pull request Nov 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant