Skip to content

Reorganize source code into smaller modules#139

Merged
attipaci merged 4 commits into
mainfrom
modularize
Mar 7, 2025
Merged

Reorganize source code into smaller modules#139
attipaci merged 4 commits into
mainfrom
modularize

Conversation

@attipaci

@attipaci attipaci commented Mar 5, 2025

Copy link
Copy Markdown
Collaborator
  • Break up large source files, and organize into meaningful functional modules.
  • Fill testing holes.
  • Fix legacy solarsystem() / solarsystem_hp() linking.
  • Minor tweaks.

@attipaci attipaci added this to the 1.3.0 milestone Mar 5, 2025
@attipaci attipaci self-assigned this Mar 5, 2025
@codecov

codecov Bot commented Mar 5, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 99.91919% with 2 lines in your changes missing coverage. Please review.

Project coverage is 99.90%. Comparing base (3be4bab) to head (e1045c6).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
src/cio.c 98.86% 0 Missing and 2 partials ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #139      +/-   ##
==========================================
+ Coverage   99.56%   99.90%   +0.33%     
==========================================
  Files           9       22      +13     
  Lines        4138     4161      +23     
  Branches      772      779       +7     
==========================================
+ Hits         4120     4157      +37     
+ Misses          4        0       -4     
+ Partials       14        4      -10     
Flag Coverage Δ
unittests 99.90% <99.91%> (+0.33%) ⬆️

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

Files with missing lines Coverage Δ
src/calendar.c 100.00% <100.00%> (ø)
src/coords.c 100.00% <100.00%> (ø)
src/earth.c 100.00% <100.00%> (ø)
src/equinox.c 100.00% <100.00%> (ø)
src/grav.c 100.00% <100.00%> (ø)
src/nutation.c 100.00% <ø> (ø)
src/observer.c 100.00% <100.00%> (ø)
src/orbital.c 100.00% <100.00%> (ø)
src/parse.c 100.00% <100.00%> (ø)
src/place.c 100.00% <100.00%> (ø)
... and 9 more

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 978f491...e1045c6. Read the comment docs.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@attipaci attipaci force-pushed the modularize branch 2 times, most recently from 88bb3d8 to 8f82477 Compare March 5, 2025 19:49
@attipaci attipaci added the enhancement New feature or request label Mar 5, 2025
@attipaci attipaci changed the title Break up super.c into a few modules Reorganize source code into smaller modules Mar 6, 2025
@attipaci attipaci force-pushed the modularize branch 19 times, most recently from 40dc8be to 653b0f9 Compare March 7, 2025 16:57
@attipaci attipaci marked this pull request as ready for review March 7, 2025 21:06
@attipaci attipaci merged commit 59d87a2 into main Mar 7, 2025
@attipaci attipaci deleted the modularize branch March 7, 2025 21:17
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