-
Notifications
You must be signed in to change notification settings - Fork 33
Pull Request Checklist
Zach Burnett edited this page Sep 9, 2024
·
6 revisions
- request a review from someone specific, to avoid making the maintainers review every PR
- add a build milestone, i.e.
24Q4_B15(use the latest build if not sure) - Does this PR change user-facing code / API? (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) - update or add relevant tests
- update relevant docstrings and / or
docs/page - start a regression test and include a link to the running job (click here for instructions)
- Do truth files need to be updated ("okified")?
- after the reviewer has approved these changes, run
okify_regteststo update the truth files
- after the reviewer has approved these changes, run
- Do truth files need to be updated ("okified")?
- write news fragment(s) in
- if a JIRA ticket exists, make sure it is resolved properly
news fragment change types...
-
changes/<PR#>.general.rst: infrastructure or miscellaneous change changes/<PR#>.docs.rstchanges/<PR#>.stpipe.rstchanges/<PR#>.associations.rstchanges/<PR#>.scripts.rstchanges/<PR#>.mosaic_pipeline.rstchanges/<PR#>.patch_match.rst
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