Skip to content

Conversation

@jimmidyson
Copy link
Member

What problem does this PR solve?:

The latest devbox-install-action removes the need for custom nix installation on self-hosted runners.

Also the nixhub.io indexing has been fixed upstream so can remove our custom installation of golangci-lint and use upstream discovered version.

Which issue(s) this PR fixes:
Fixes #

How Has This Been Tested?:

Special notes for your reviewer:

The latest devbox-install-action removes the need for custom nix
installation on self-hosted runners.

Also the nixhub.io indexing has been fixed upstream so can remove
our custom installation of golangci-lint and use upstream discovered
version.
Copy link
Contributor

@dkoshkin dkoshkin left a comment

Choose a reason for hiding this comment

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

Thanks

@jimmidyson jimmidyson enabled auto-merge (squash) September 26, 2025 17:48
@jimmidyson jimmidyson merged commit 6a8fa62 into main Sep 26, 2025
24 checks passed
@jimmidyson jimmidyson deleted the jimmi/devbox-action-remove-nix branch September 26, 2025 23:46
vijayaraghavanr31 pushed a commit that referenced this pull request Oct 8, 2025
)

**What problem does this PR solve?**:

The latest devbox-install-action removes the need for custom nix
installation on self-hosted runners.

Also the nixhub.io indexing has been fixed upstream so can remove our
custom installation of golangci-lint and use upstream discovered
version.

**Which issue(s) this PR fixes**:
Fixes #

**How Has This Been Tested?**:
<!--
Please describe the tests that you ran to verify your changes.
Provide output from the tests and any manual steps needed to replicate
the tests.
-->

**Special notes for your reviewer**:
<!--
Use this to provide any additional information to the reviewers.
This may include:
- Best way to review the PR.
- Where the author wants the most review attention on.
- etc.
-->
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.

4 participants