Skip to content

chore:replace setup-zig action since it was archived.#131

Merged
gballet merged 2 commits intoblockblaz:mainfrom
GrapeBaBa:replace_setup_zig
Sep 2, 2025
Merged

chore:replace setup-zig action since it was archived.#131
gballet merged 2 commits intoblockblaz:mainfrom
GrapeBaBa:replace_setup_zig

Conversation

@GrapeBaBa
Copy link
Copy Markdown
Member

Replace setup-zig action to mlugg/setup-zig since korandoru/setup-zig was archived.

Signed-off-by: Chen Kai <281165273grape@gmail.com>
Copilot AI review requested due to automatic review settings August 31, 2025 04:27
@gemini-code-assist
Copy link
Copy Markdown

Important

Installation incomplete: to start using Gemini Code Assist, please ask the organization owner(s) to visit the Gemini Code Assist Admin Console and sign the Terms of Services.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR replaces the archived korandoru/setup-zig GitHub Action with the maintained mlugg/setup-zig action across CI workflows.

Key changes:

  • Updated action reference from korandoru/setup-zig@v1 to mlugg/setup-zig@v2.0.5
  • Changed parameter name from zig-version to version to match new action's API
  • Updated Zig versions (0.14.0 → 0.14.0, 0.14.1 → 0.14.1) to maintain consistency

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/risc0.yml Updates setup-zig action and parameter naming
.github/workflows/ci.yml Updates setup-zig action across multiple jobs with parameter naming changes

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@gballet
Copy link
Copy Markdown
Contributor

gballet commented Aug 31, 2025

I'm getting a lot of cache misses and inability to download the compiler :(

Copy link
Copy Markdown
Contributor

@gballet gballet left a comment

Choose a reason for hiding this comment

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

I'm not enclined to merge this at this time, since the previous one is still working with zig 0.14.0. I'm going to sit on this one for a couple of days.

@GrapeBaBa
Copy link
Copy Markdown
Member Author

I'm getting a lot of cache misses and inability to download the compiler :(

That is what I met when using previous one and make this PR.

@gballet
Copy link
Copy Markdown
Contributor

gballet commented Sep 2, 2025

Yeah but that's not why both of these fails. That looked like an issue on the compiler hosting side. It seems to be fixed now though.

@gballet gballet merged commit 64ee736 into blockblaz:main Sep 2, 2025
12 of 15 checks passed
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