-
Notifications
You must be signed in to change notification settings - Fork 32
Update tox environments #1540
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
base: main
Are you sure you want to change the base?
Update tox environments #1540
Conversation
e022e98 to
ec7995e
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1540 +/- ##
=======================================
Coverage 81.02% 81.02%
=======================================
Files 136 136
Lines 8063 8063
=======================================
Hits 6533 6533
Misses 1530 1530 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
mairanteodoro
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
ec7995e to
57f2b51
Compare
57f2b51 to
722411f
Compare
722411f to
b08965a
Compare
|
This kind of tox change looks reasonable to me but I don't think a lot about it; I'm happy to defer to you and Zach. I'll go ahead and approve and let you merge when you see appropriate? I note that a bunch of tests are failing now, maybe just because this is old; sorry for having dropped this. |
b08965a to
38a989c
Compare
0c5b19f to
efcb323
Compare
efcb323 to
cb24b47
Compare
This PR cleans up the tox environments in
romancalto match the changes made to those inradandroman_datamodels.Tasks
24Q4_B15(use the latest build if not sure)no-changelog-entry-needed)changes/:echo "changed something" > changes/<PR#>.<changetype>.rst(see below for change types)docs/pageokify_regteststo update the truth filesnews fragment change types...
changes/<PR#>.general.rst: infrastructure or miscellaneous changechanges/<PR#>.docs.rstchanges/<PR#>.stpipe.rstchanges/<PR#>.associations.rstchanges/<PR#>.scripts.rstchanges/<PR#>.mosaic_pipeline.rstchanges/<PR#>.patch_match.rststeps
changes/<PR#>.dq_init.rstchanges/<PR#>.saturation.rstchanges/<PR#>.refpix.rstchanges/<PR#>.linearity.rstchanges/<PR#>.dark_current.rstchanges/<PR#>.jump_detection.rstchanges/<PR#>.ramp_fitting.rstchanges/<PR#>.assign_wcs.rstchanges/<PR#>.flatfield.rstchanges/<PR#>.photom.rstchanges/<PR#>.flux.rstchanges/<PR#>.source_detection.rstchanges/<PR#>.tweakreg.rstchanges/<PR#>.skymatch.rstchanges/<PR#>.outlier_detection.rstchanges/<PR#>.resample.rstchanges/<PR#>.source_catalog.rst