Skip to content

Bump nodeenv to 1.11.1 so the lint task runs again - #7355

Merged
marco-c merged 1 commit into
mozilla:masterfrom
RizgarOzan:fix/nodeenv-1.11.1
Oct 30, 2026
Merged

marco-c merged 1 commit into
mozilla:masterfrom
RizgarOzan:fix/nodeenv-1.11.1

Conversation

@RizgarOzan

Copy link
Copy Markdown
Contributor

Fixes #7353

What: bumps nodeenv 1.10.0 -> 1.11.1 in uv.lock. Nothing else changes.

Why: nodejs.org's release index now lists v27.0.0-alpha.0.0.0 first. nodeenv 1.10.0 tries to int() that when it picks a Node version for the prettier hook and dies with invalid literal for int() with base 10: '0-alpha', so every lint task fails before pre-commit runs anything. nodeenv 1.11.1 (released yesterday) skips pre-releases: ekalinin/nodeenv#454.

How I tested: ran python -m nodeenv --prebuilt <dir> with both versions. 1.10.0 reproduces the exact traceback from the CI log, 1.11.1 installs Node 26.13.0 fine. The lock was regenerated with uv lock --upgrade-package nodeenv on uv 0.13.6 so the lock revision stays the same.

nodeenv 1.10.0 chokes on the new v27.0.0-alpha entry in the Node.js
release index while picking a version for the prettier hook, so every
lint task in CI dies before pre-commit even starts. 1.11.1 skips
pre-releases.

Fixes mozilla#7353
@RizgarOzan
RizgarOzan requested a review from a team as a code owner October 30, 2026 12:24
Copilot AI balanced review requested due to automatic review settings October 30, 2026 12:24

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.

Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.

@marco-c
marco-c merged commit da2f1e6 into mozilla:master Oct 30, 2026
12 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.

The linting task is failing with invalid literal for int() with base 10: '0-alpha'

3 participants