Skip to content

DOC: Fix resample docstring#561

Merged
braingram merged 3 commits into
spacetelescope:mainfrom
pllim:doc-resample-for-inheritance
Jul 14, 2026
Merged

DOC: Fix resample docstring#561
braingram merged 3 commits into
spacetelescope:mainfrom
pllim:doc-resample-for-inheritance

Conversation

@pllim

@pllim pllim commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Need this for spacetelescope/jwst#10711

No need for change log nor RT.

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 changelog readme for instructions)
    • if your change breaks existing functionality, also add a changes/<PR#>.breaking.rst news fragment
  • run regression tests with this branch installed ("git+https://github.com/<fork>/stcal@<branch>")

so jwst can inherit it without complain
@codecov

codecov Bot commented Jul 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.90%. Comparing base (0a0b9e7) to head (dad031c).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #561   +/-   ##
=======================================
  Coverage   91.90%   91.90%           
=======================================
  Files          63       63           
  Lines        8926     8926           
=======================================
  Hits         8203     8203           
  Misses        723      723           

☔ View full report in Codecov by Harness.
📢 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 July 14, 2026 02:36
@pllim
pllim requested a review from a team as a code owner July 14, 2026 02:36
@braingram
braingram requested a review from mcara July 14, 2026 10:46

@mcara mcara left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@braingram
braingram merged commit bb256db into spacetelescope:main Jul 14, 2026
26 checks passed
@pllim
pllim deleted the doc-resample-for-inheritance branch July 14, 2026 15:34
@pllim

pllim commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

Thanks!

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