Skip to content

Conversation

@ryoppippi
Copy link
Member

@ryoppippi ryoppippi commented Jan 22, 2026

Summary

  • Use install-nix-action directly instead of setup-nix composite action
  • Remove skip-mock-server option (not needed with nix develop)
  • Leverage the full devShell environment instead of manually installing individual tools

Test plan

  • Verify next release workflow runs successfully

Summary by cubic

Simplified the release workflow to run tasks with nix develop while keeping the shared setup-nix composite action. This standardizes tooling and relies on the devShell for dependencies.

  • Refactors
    • Kept the local setup-nix composite action; run commands via nix develop.
    • Replaced skip-mock-server with skip-uv-sync since the devShell covers dependency setup.

Written for commit f7e55dd. Summary will update on new commits.

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

Use the setup-nix composite action for consistent Nix setup across
workflows, while still using nix develop for commands to leverage the
full devShell environment.

- Replace direct install-nix-action with setup-nix composite action
- Add skip-uv-sync since nix develop handles dependencies
- Keep nix develop --command for build commands
@ryoppippi ryoppippi force-pushed the fix/release-workflow-nix-develop branch from f1eb7a1 to f7e55dd Compare January 22, 2026 10:55
@ryoppippi ryoppippi merged commit af5f66e into main Jan 22, 2026
14 checks passed
@ryoppippi ryoppippi deleted the fix/release-workflow-nix-develop branch January 22, 2026 10:55
Copy link
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 updates the release workflow to install Nix directly via cachix/install-nix-action and rely on the project’s devShell via nix develop instead of the repository’s setup-nix composite action.

Changes:

  • Replace ./.github/actions/setup-nix usage in the release workflow with cachix/install-nix-action.
  • Remove the skip-mock-server input (no longer applicable when not using the composite action).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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