Skip to content

Conversation

pllim
Copy link
Collaborator

@pllim pllim commented Apr 8, 2025

This PR forward-ports

to keep main content somewhat in-sync with latest release so the former does not appear outdated.

Tasks

  • If you have a specific reviewer in mind, tag them.
  • add a build milestone, i.e. Build 12.0 (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 changelog readme for instructions)
    • 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_regtests to update the truth files
  • if a JIRA ticket exists, make sure it is resolved properly

@pllim

This comment was marked as resolved.

Copy link

codecov bot commented Apr 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.40%. Comparing base (e4916d3) to head (5f0dc94).
Report is 663 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9382      +/-   ##
==========================================
- Coverage   75.40%   75.40%   -0.01%     
==========================================
  Files         368      368              
  Lines       36839    36837       -2     
==========================================
- Hits        27779    27777       -2     
  Misses       9060     9060              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@pllim pllim marked this pull request as ready for review April 8, 2025 21:56
@pllim pllim requested a review from a team as a code owner April 8, 2025 21:56
@pllim pllim requested a review from tapastro April 8, 2025 21:57
@pllim pllim force-pushed the forwardport-jhunk-1.18.0 branch from 826c265 to 41c4e05 Compare April 9, 2025 17:07
Remove changelog fragments like spacetelescope#9354 except the one whose backport was reverted
@pllim pllim force-pushed the forwardport-jhunk-1.18.0 branch from 41c4e05 to fda0180 Compare April 9, 2025 17:10
@pllim
Copy link
Collaborator Author

pllim commented Apr 9, 2025

Okay I removed the fragments. I think this is ready to merge unless you catch something else.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This update is not needed if #9386 is merged first.

@pllim pllim merged commit d8604ee into spacetelescope:main Apr 9, 2025
29 checks passed
@pllim pllim deleted the forwardport-jhunk-1.18.0 branch April 9, 2025 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants