Skip to content

ci: use nix dependencies in ci jobs#31

Merged
danielvcorreia merged 4 commits into
wavecomtech:mainfrom
danielvcorreia:feat/ci-deps-nix
Jun 25, 2026
Merged

ci: use nix dependencies in ci jobs#31
danielvcorreia merged 4 commits into
wavecomtech:mainfrom
danielvcorreia:feat/ci-deps-nix

Conversation

@danielvcorreia

Copy link
Copy Markdown
Contributor

Context

The check-code-regeneration job was failing because the local development Nix setup was using go1.24.3 and the job was using the hardcoded go1.21. By using Nix dependencies in the ci jobs, problems like these should not occur again

This should close #27

@danielvcorreia danielvcorreia self-assigned this Jun 16, 2026

Copilot AI left a comment

Copy link
Copy Markdown

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 CI workflow to run generation/build/test inside the repository’s Nix dev shell so CI uses the same Go/tooling versions as local Nix development, addressing workflow breakage due to Go version drift (issue #27).

Changes:

  • Replace actions/setup-go + manual go install steps with DeterminateSystems/nix-installer-action.
  • Run go generate, go build, and go test via nix develop -c ... to use flake-provided tooling.

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

Comment thread .github/workflows/ci.yml

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread .github/workflows/ci.yml Outdated
@danielvcorreia danielvcorreia merged commit bb1e607 into wavecomtech:main Jun 25, 2026
2 of 3 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.

Github workflow ci is breaking

3 participants