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

MNT: Bump required deps, update workflows #341

Merged
merged 3 commits into from
Nov 29, 2022

Conversation

pllim
Copy link
Contributor

@pllim pllim commented Nov 29, 2022

Description

This pull request is to go with spacetelescope/stsynphot_refactor#167

and remove old code. Also update CI matrix
@codecov
Copy link

codecov bot commented Nov 29, 2022

Codecov Report

Base: 94.46% // Head: 97.08% // Increases project coverage by +2.61% 🎉

Coverage data is based on head (b4fd152) compared to base (cb3ff4b).
Patch coverage: 91.30% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #341      +/-   ##
==========================================
+ Coverage   94.46%   97.08%   +2.61%     
==========================================
  Files          17       17              
  Lines        2096     2024      -72     
==========================================
- Hits         1980     1965      -15     
+ Misses        116       59      -57     
Impacted Files Coverage Δ
synphot/compat.py 88.23% <85.71%> (+8.23%) ⬆️
synphot/filter_parameterization/filter_fft.py 98.27% <100.00%> (+1.55%) ⬆️
synphot/models.py 95.75% <100.00%> (+10.94%) ⬆️
synphot/spectrum.py 99.49% <100.00%> (+0.16%) ⬆️
synphot/units.py 97.54% <100.00%> (+5.81%) ⬆️
synphot/utils.py 99.00% <0.00%> (+1.00%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@pllim pllim merged commit a5fa653 into spacetelescope:master Nov 29, 2022
@pllim pllim deleted the update-ci branch November 29, 2022 20:37
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant