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

Separate super.c source module. #60

Merged
merged 4 commits into from
Aug 24, 2024
Merged

Separate super.c source module. #60

merged 4 commits into from
Aug 24, 2024

Conversation

attipaci
Copy link
Collaborator

Add new super.c with SuperNOVAS only functions that are not required by novas.c. With it we reduce the bloating of the already large novas.c source a little. At the same time we continue to support linking against novas.o for legacy NOVAS C applications, which might want it.

@attipaci attipaci added the enhancement New feature or request label Aug 24, 2024
@attipaci attipaci added this to the 1.1.1 milestone Aug 24, 2024
@attipaci attipaci self-assigned this Aug 24, 2024
Copy link

codecov bot commented Aug 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.64%. Comparing base (2ac06d7) to head (1593f6a).
Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #60   +/-   ##
=======================================
  Coverage   99.64%   99.64%           
=======================================
  Files           6        7    +1     
  Lines        2793     2793           
  Branches      539      539           
=======================================
  Hits         2783     2783           
  Partials       10       10           
Flag Coverage Δ
unittests 99.64% <100.00%> (ø)

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

Files Coverage Δ
src/novas.c 99.54% <100.00%> (-0.05%) ⬇️
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 d1ad032...1593f6a. Read the comment docs.

@attipaci attipaci marked this pull request as ready for review August 24, 2024 12:30
@attipaci attipaci merged commit 16f6865 into main Aug 24, 2024
13 checks passed
@attipaci attipaci deleted the modular-super branch August 24, 2024 12:51
attipaci added a commit that referenced this pull request Aug 24, 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