Skip to content

CI: Replace Garnix with GHA#396

Open
9999years wants to merge 1 commit into
mainfrom
wiggles/lmsr
Open

CI: Replace Garnix with GHA#396
9999years wants to merge 1 commit into
mainfrom
wiggles/lmsr

Conversation

@9999years

@9999years 9999years commented Mar 12, 2026

Copy link
Copy Markdown
Member

@9999years 9999years requested a review from a team as a code owner March 12, 2026 22:18
@9999years 9999years requested review from dtpowl and parsonsmatt March 12, 2026 22:18
Comment thread .github/workflows/ci.yaml
Comment thread .github/workflows/ci.yaml Outdated
Comment thread garnix.yaml Outdated
Comment thread .github/workflows/ci.yaml
- name: Checkout code
uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0

- uses: cachix/install-nix-action@7be5dee1421f63d07e71ce6e0a9f8a4b07c2a487 # v31.6.1

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Would a caching step after this one be beneficial?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes, but we don't have a public/OSS Nix cache yet.

See: https://linear.app/mercury/issue/OSPO-23/oss-nix-cache

Comment thread .github/workflows/ci.yaml
github_access_token: ${{ secrets.GITHUB_TOKEN }}
extra_nix_config: |
extra-experimental-features = nix-command flakes
accept-flake-config = true

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
accept-flake-config = true

Isn't that setting a no-op since the nixConfig was removed from flake.nix?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yeah, but hopefully we'll have more config (for a new cache) in the future.

@9999years 9999years force-pushed the wiggles/lmsr branch 2 times, most recently from 1e7afc0 to dfce171 Compare June 15, 2026 18:52
@9999years 9999years enabled auto-merge (squash) June 16, 2026 17:26
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.

3 participants