Skip to content

use CRDS contexts workflow#9998

Draft
zacharyburnett wants to merge 1 commit into
spacetelescope:mainfrom
zacharyburnett:crds_workflow
Draft

use CRDS contexts workflow#9998
zacharyburnett wants to merge 1 commit into
spacetelescope:mainfrom
zacharyburnett:crds_workflow

Conversation

@zacharyburnett

Copy link
Copy Markdown
Collaborator

move back to the centralized CRDS reusable workflow for retrieving contexts; we'll need to adapt the calls in tests.yml to use jwst-edit as well

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

@zacharyburnett zacharyburnett added this to the Build 12.1.1 milestone Nov 6, 2025
@zacharyburnett zacharyburnett self-assigned this Nov 6, 2025
@github-actions github-actions Bot added the automation Continuous Integration (CI) and testing automation tools label Nov 6, 2025
@codecov

codecov Bot commented Nov 6, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.77%. Comparing base (61bd2fe) to head (0222363).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #9998   +/-   ##
=======================================
  Coverage   85.77%   85.77%           
=======================================
  Files         372      372           
  Lines       40052    40052           
=======================================
  Hits        34353    34353           
  Misses       5699     5699           

☔ 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.

@zacharyburnett zacharyburnett marked this pull request as ready for review April 10, 2026 15:58
@zacharyburnett zacharyburnett enabled auto-merge (squash) April 10, 2026 15:58
- linux: check-dependencies
latest_crds_contexts:
uses: ./.github/workflows/contexts.yml
uses: spacetelescope/crds/.github/workflows/contexts.yml@main

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Should pin to hash?

@melanieclarke

Copy link
Copy Markdown
Collaborator

@zacharyburnett - what's the status of this PR?

@zacharyburnett zacharyburnett marked this pull request as draft May 29, 2026 13:32
auto-merge was automatically disabled May 29, 2026 13:32

Pull request was converted to draft

@zacharyburnett

Copy link
Copy Markdown
Collaborator Author

@zacharyburnett - what's the status of this PR?

I'm working on simplifying the CRDS reusable workflow into an action we can use in a step, but in the meantime we should also be pinning to a specific commit hash so that changes to crds@main don't affect downstream CI; I'll work on this today

@zacharyburnett

Copy link
Copy Markdown
Collaborator Author

after talking with @braingram we've decided to move this to a more maintainable version in the upcoming package template, so we'll leave this as-is until then

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automation Continuous Integration (CI) and testing automation tools no-changelog-entry-needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants