Skip to content

Fix nix build command - #377

Open
sequencer wants to merge 1 commit into
indoorvivants:mainfrom
sequencer:nix-build-fix
Open

Fix nix build command#377
sequencer wants to merge 1 commit into
indoorvivants:mainfrom
sequencer:nix-build-fix

Conversation

@sequencer

Copy link
Copy Markdown
  • Fix Nix build failure caused by missing scala3-sbt-bridge:3.7.3 dependency
  • Add bindgen/compile to depsWarmupCommand to fetch Scala 3.7.3 compiler bridge during dependency phase

Copilot AI review requested due to automatic review settings December 31, 2025 23:45

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 fixes a Nix build failure caused by a missing scala3-sbt-bridge:3.7.3 dependency by adding the bindgen/compile command to the dependency warmup phase and updating the dependency hash accordingly. It also adds CI validation to ensure future Nix builds remain functional.

  • Adds bindgen/compile to depsWarmupCommand in flake.nix to fetch the Scala 3.7.3 compiler bridge during the dependency phase
  • Updates depsSha256 hash to reflect the new dependency set
  • Introduces a new nix_build CI job to validate Nix builds and prevent future regressions

Reviewed changes

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

File Description
flake.nix Adds bindgen/compile to dependency warmup commands and updates dependency hash to include Scala 3.7.3 compiler bridge
.github/workflows/ci.yml Adds new nix_build CI job to validate Nix builds and integrates it into the Mergify checkpoint

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

@sequencer
sequencer force-pushed the nix-build-fix branch 2 times, most recently from 098d869 to 5c81318 Compare January 1, 2026 15:56
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