I also removed the py311 environment (testing with python 3.11) since we're already running py311-cov (python 3.11 while measuring coverage). This change will require updates to the branch protection rules as currently py311 is marked as required. I'm happy to remove that change from this PR (if that's preferred) just let me know.
Tasks
[ ] update or add relevant tests
[ ] update relevant docstrings and / or docs/ page
[ ] Does this PR change any API used downstream? (if not, label with no-changelog-entry-needed)
[ ] write news fragment(s) in changes/: echo "changed something" > changes/<PR#>.<changetype>.rst (see below for change types)
news fragment change types...
- ``changes/.feature.rst``: new feature
- ``changes/.bugfix.rst``: fixes an issue
- ``changes/.doc.rst``: documentation change
- ``changes/.removal.rst``: deprecation or removal of public API
- ``changes/.misc.rst``: infrastructure or miscellaneous change
This will need the branch protections updated so that the linux py311 requirement is replaced by the linux py311-cov job instead. @zacharyburnett or @braingram can either of you update the branch protections after merging this?
This PR adds python 3.13 to the CI.
I also removed the
py311
environment (testing with python 3.11) since we're already runningpy311-cov
(python 3.11 while measuring coverage). This change will require updates to the branch protection rules as currentlypy311
is marked as required. I'm happy to remove that change from this PR (if that's preferred) just let me know.Tasks
docs/
pageno-changelog-entry-needed
)changes/
:echo "changed something" > changes/<PR#>.<changetype>.rst
(see below for change types)romancal
regression test with this branch installed ("git+https://github.com/<fork>/roman_datamodels@<branch>"
)news fragment change types...
- ``changes/Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Additional details and impacted files
```diff @@ Coverage Diff @@ ## main #401 +/- ## ========================================== + Coverage 97.56% 97.75% +0.19% ========================================== Files 30 36 +6 Lines 2788 3434 +646 ========================================== + Hits 2720 3357 +637 - Misses 68 77 +9 ```:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This will need the branch protections updated so that the linux py311 requirement is replaced by the linux py311-cov job instead. @zacharyburnett or @braingram can either of you update the branch protections after merging this?
Thanks! I updated the branch protection rules. Let me know if they look good to you or if you notice any other changes there that would be helpful.