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

Version info #37

Merged
merged 4 commits into from
Jun 3, 2021
Merged

Version info #37

merged 4 commits into from
Jun 3, 2021

Conversation

ConorMacBride
Copy link
Owner

Add version info string at mcalf.version.version, using generated _version.py, or setuptools_scm for dev installs. Version string is included in mcalf.models.FitResults.save files as a VERSION header in the primary HDU.

Signed-off-by: Conor MacBride <conor@macbride.me>
Signed-off-by: Conor MacBride <conor@macbride.me>
Signed-off-by: Conor MacBride <conor@macbride.me>
@codecov
Copy link

codecov bot commented Jun 3, 2021

Codecov Report

Merging #37 (cb23c40) into main (8e202da) will decrease coverage by 0.05%.
The diff coverage is 85.71%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #37      +/-   ##
==========================================
- Coverage   94.49%   94.44%   -0.06%     
==========================================
  Files          16       17       +1     
  Lines        1217     1224       +7     
==========================================
+ Hits         1150     1156       +6     
- Misses         67       68       +1     
Impacted Files Coverage Δ
src/mcalf/version.py 83.33% <83.33%> (ø)
src/mcalf/models/results.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Signed-off-by: Conor MacBride <conor@macbride.me>
@ConorMacBride ConorMacBride merged commit 12bb041 into main Jun 3, 2021
@ConorMacBride ConorMacBride deleted the version-info branch June 3, 2021 15:25
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant