Skip to content

Extract NVHPC and pixi setup into reusable composite action#81

Merged
joshkamm merged 2 commits intomainfrom
ci-composite-action
Mar 5, 2026
Merged

Extract NVHPC and pixi setup into reusable composite action#81
joshkamm merged 2 commits intomainfrom
ci-composite-action

Conversation

@joshkamm
Copy link
Member

@joshkamm joshkamm commented Mar 4, 2026

Summary

  • Extracts the shared build environment setup (pixi version extraction, pixi install, NVHPC cache, NVHPC install) from slater-gpu-base.yml into a reusable composite action at .github/actions/setup-build-env/
  • Moves install_nvhpc.sh into the composite action directory so it's co-located with the action that uses it
  • Updates slater-gpu-base.yml to use the new composite action via a relative path (./.github/actions/setup-build-env)

This allows downstream repos (e.g. ZEST) to reference the same setup environment with a single uses: ZimmermanGroup/SlaterGPU/.github/actions/setup-build-env@main line, avoiding duplication of the NVHPC + pixi infrastructure.

Test plan

  • CI passes on this branch (SlaterGPU build + tests still work after refactor)

🤖 Generated with Claude Code

joshkamm and others added 2 commits March 4, 2026 10:12
Moves the shared build environment setup (pixi version extraction,
pixi install, NVHPC cache, NVHPC install) out of slater-gpu-base.yml
and into a reusable composite action at .github/actions/setup-build-env/.
This allows downstream repos (e.g. ZEST) to use the same setup with
a single 'uses:' line without duplicating infrastructure.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
@joshkamm joshkamm self-assigned this Mar 4, 2026
@joshkamm joshkamm marked this pull request as ready for review March 4, 2026 16:14
@joshkamm joshkamm requested a review from DuttaS12 March 4, 2026 16:24
@joshkamm joshkamm merged commit 4aa3bba into main Mar 5, 2026
1 check passed
@joshkamm joshkamm deleted the ci-composite-action branch March 5, 2026 03:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants