Skip to content

util: fix -0 formatting when numericSeparator is enabled - #63815

Closed
watilde wants to merge 1 commit into
nodejs:mainfrom
watilde:utils-fix
Closed

util: fix -0 formatting when numericSeparator is enabled#63815
watilde wants to merge 1 commit into
nodejs:mainfrom
watilde:utils-fix

Conversation

@watilde

Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Daijiro Wachi <daijiro.wachi@gmail.com>
@nodejs-github-botnodejs-github-bot added needs-ci PRs that need a full CI run. util Issues and PRs related to the built-in util module. labels Jun 9, 2026
@lpincalpinca added the request-ci Add this label to start a Jenkins CI on a PR. label Jun 9, 2026
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 9, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@watildewatilde added the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 12, 2026
@nodejs-github-botnodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Jun 12, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/63815
✔ Done loading data for nodejs/node/pull/63815
----------------------------------- PR info ------------------------------------
Title util: fix -0 formatting when numericSeparator is enabled (#63815)
Author Daijiro Wachi <daijiro.wachi@gmail.com> (@watilde)
Branch watilde:utils-fix -> nodejs:main
Labels util, needs-ci
Commits 1
- util: fix -0 formatting when numericSeparator is enabled
Committers 1
- Daijiro Wachi <daijiro.wachi@gmail.com>
PR-URL: https://github.com/nodejs/node/pull/63815
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jordan Harband <ljharb@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/63815
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jordan Harband <ljharb@gmail.com>
--------------------------------------------------------------------------------
ℹ This PR was created on Tue, 09 Jun 2026 10:34:34 GMT
✔ Approvals: 2
✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/63815#pullrequestreview-4462290077
✔ - Jordan Harband (@ljharb): https://github.com/nodejs/node/pull/63815#pullrequestreview-4462431386
✘ GitHub CI failed with status: FAILURE
ℹ Last Full PR CI on 2026-06-09T21:08:36Z: https://ci.nodejs.org/job/node-test-pull-request/74005/
- Querying data for job/node-test-pull-request/74005/
✔ Build data downloaded
- Querying failures of job/node-test-commit/88553/
✔ Data downloaded
✘ 1 failure(s) on the last Jenkins CI run
--------------------------------------------------------------------------------
✔ Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/27404596390

@watildewatilde added request-ci Add this label to start a Jenkins CI on a PR. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Jun 13, 2026
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 13, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

watilde added a commit that referenced this pull request Jun 15, 2026
Signed-off-by: Daijiro Wachi <daijiro.wachi@gmail.com>
PR-URL: #63815
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jordan Harband <ljharb@gmail.com>
@watilde

Copy link
Copy Markdown
MemberAuthor

Landed in 3ad2463

@watildewatilde closed this Jun 15, 2026
aduh95 pushed a commit that referenced this pull request Jun 18, 2026
Signed-off-by: Daijiro Wachi <daijiro.wachi@gmail.com>
PR-URL: #63815
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jordan Harband <ljharb@gmail.com>
aduh95 pushed a commit that referenced this pull request Jun 25, 2026
Signed-off-by: Daijiro Wachi <daijiro.wachi@gmail.com>
PR-URL: #63815
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jordan Harband <ljharb@gmail.com>
aduh95 pushed a commit that referenced this pull request Jul 22, 2026
Signed-off-by: Daijiro Wachi <daijiro.wachi@gmail.com>
PR-URL: #63815
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jordan Harband <ljharb@gmail.com>
aduh95 pushed a commit that referenced this pull request Jul 30, 2026
Signed-off-by: Daijiro Wachi <daijiro.wachi@gmail.com>
PR-URL: #63815
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jordan Harband <ljharb@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-ciPRs that need a full CI run.utilIssues and PRs related to the built-in util module.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@watilde@nodejs-github-bot@ljharb@lpinca