Skip to content

Fix Nix build issue#390

Merged
RoseSecurity merged 3 commits intoRoseSecurity:mainfrom
ApprenticeofEnder:main
Feb 7, 2026
Merged

Fix Nix build issue#390
RoseSecurity merged 3 commits intoRoseSecurity:mainfrom
ApprenticeofEnder:main

Conversation

@ApprenticeofEnder
Copy link
Contributor

@ApprenticeofEnder ApprenticeofEnder commented Feb 6, 2026

Why

  • Nix flake wasn't building properly, now it is

What

  • Updated the go.sum file to reference hashicorp/copywrite
  • Updated vendor hash in default.nix because Nix

References

Summary by CodeRabbit

  • Chores
    • Updated project dependencies to latest versions.
    • Applied minor code formatting adjustments for consistency.

Signed-off-by: Robert Babaev <github@robertbabaev.tech>
Signed-off-by: Robert Babaev <github@robertbabaev.tech>
Signed-off-by: Robert Babaev <github@robertbabaev.tech>
@CLAassistant
Copy link

CLAassistant commented Feb 6, 2026

CLA assistant check
All committers have signed the CLA.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 6, 2026

📝 Walkthrough

Walkthrough

This PR updates the Nix build configuration for Terramaid with a refreshed vendorHash and minor formatting adjustments to align the subPackages and ldflags attributes. The vendorHash change resolves the missing go.sum entry issue reported in the linked issue.

Changes

Cohort / File(s) Summary
Nix Build Configuration
default.nix
Updated vendorHash from sha256-CfUrdpNkjkmXO09eyDDWmYNy7vDGXxQCiKmPI3uo96s= to sha256-ZnYGyoKYnx9inSA2FryFds5jAf3L45nAsgm5ElXtv9Y= to resolve missing go.sum dependencies. Simplified ldflags formatting to a single quoted list and aligned subPackages quoting style. Minor whitespace adjustments for consistency.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

Suggested labels

patch

🚥 Pre-merge checks | ✅ 4 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Fix Nix build issue' is vague and generic, failing to specify which Nix build issue was fixed or what changes were made. Consider a more specific title like 'Fix Nix build by updating vendorHash and adding missing go.sum entry' to clearly convey the primary changes.
✅ Passed checks (4 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The pull request addresses issue #389 by adding the missing go.sum entry for github.com/hashicorp/copywrite and updating the vendorHash, resolving the Nix build failure.
Out of Scope Changes check ✅ Passed All changes in default.nix (vendorHash update, formatting adjustments, ldflags simplification) are directly related to fixing the Nix build issue described in issue #389.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ApprenticeofEnder
Copy link
Contributor Author

Regarding the formatting, I use alejandra on Nix by default, but that's just personal preference. Feel free to format using nixpkgs-fmt since it looks like that's what you've used elsewhere?

Copy link
Owner

@RoseSecurity RoseSecurity left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@RoseSecurity
Copy link
Owner

The Nix package was implemented by another community member. I personally don't use it, so I appreciate this fix even more

@RoseSecurity RoseSecurity merged commit d356d83 into RoseSecurity:main Feb 7, 2026
8 checks passed
@ApprenticeofEnder
Copy link
Contributor Author

The Nix package was implemented by another community member. I personally don't use it, so I appreciate this fix even more

No worries, figure this will help adoption among the other Nix-brained folks in the community :D

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.

Nix build fails due to missing go.sum entry?

3 participants