NatLabRockies / NatLabRockies/GridAnalysisToolkit
Wire the Sienna v5 fixture into CI
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 1
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
The fixture-refresh workflow already offers v5 (Manifest.v5.toml and make sienna-fixture-v5 exist), but tests.yml only exercises v4 — test_sienna_regression.py has no v5 coverage and the GAT_SIENNA_V5_FIXTURE env var is exported to pytest without anything consuming it. Add v5 regression tests and the corresponding cache/fail-fast wiring in tests.yml, mirroring the v4 setup.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Compare the v4 setup in tests.yml with test_sienna_regression.py, then trace how GAT_SIENNA_V5_FIXTURE is passed to pytest. Add matching v5 regression coverage and the cache/fail-fast wiring in tests.yml; done means the v5 tests run in CI and the fixture variable is consumed.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- github-actions, python
- Domain
- ci-cd, testing
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Clearly specified
- Newbie friendliness
- 78/100