Skip to content

tools: fix timeout errors in lint-nix job#62265

Merged
nodejs-github-bot merged 4 commits intonodejs:mainfrom
aduh95:fix-lint-nix-timeout
Mar 17, 2026
Merged

tools: fix timeout errors in lint-nix job#62265
nodejs-github-bot merged 4 commits intonodejs:mainfrom
aduh95:fix-lint-nix-timeout

Conversation

@aduh95
Copy link
Contributor

@aduh95 aduh95 commented Mar 15, 2026

Fixes: #62219

@nodejs-github-bot nodejs-github-bot added the tools Issues and PRs related to the tools directory. label Mar 15, 2026
@aduh95 aduh95 requested a review from Renegade334 March 16, 2026 18:20
Copy link
Member

@Renegade334 Renegade334 left a comment

Choose a reason for hiding this comment

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

update-nixpkgs-pin won't remove this, so it might be worth a TODO comment to remove once the patch makes it to a release.

@aduh95
Copy link
Contributor Author

aduh95 commented Mar 16, 2026

it might be worth a TODO comment to remove once the patch makes it to a release.

Note that this is a simple revert, not exactly a patch, so it's unlikely to land upstream (I would expect they land a proper fix). If the nixpkgs pin updates the version of treefmt and the commit no longer reverts cleanly (which would likely happen when a proper fix (or even a revert) lands), we should see the lint-nix job fail.

@aduh95 aduh95 added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Mar 16, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Mar 17, 2026
@nodejs-github-bot nodejs-github-bot merged commit 3db2206 into nodejs:main Mar 17, 2026
237 of 245 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in 3db2206

@aduh95 aduh95 deleted the fix-lint-nix-timeout branch March 17, 2026 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. tools Issues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

lint-nix action is currently unreliable

5 participants