src: remove unused variables from report by iknoom · Pull Request #60047 · nodejs/node · GitHub
Skip to content

src: remove unused variables from report#60047

Merged
nodejs-github-bot merged 1 commit intonodejs:mainfrom
iknoom:remove-unused-variable
Oct 2, 2025
Merged

src: remove unused variables from report#60047
nodejs-github-bot merged 1 commit intonodejs:mainfrom
iknoom:remove-unused-variable

Conversation

@iknoom
Copy link
Copy Markdown
Contributor

@iknoom iknoom commented Sep 28, 2025

This PR removes unused variables in node_report.cc detected by clang-tidy(bugprone-unused-local-non-trivial-variable).

@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. report Issues and PRs related to process.report. labels Sep 28, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 28, 2025

@theanarkh theanarkh added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 28, 2025
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 28, 2025
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@addaleax addaleax added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. labels Sep 28, 2025
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 28, 2025
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@addaleax addaleax added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 2, 2025
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 2, 2025
@nodejs-github-bot nodejs-github-bot merged commit 36b1cc2 into nodejs:main Oct 2, 2025
81 checks passed
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

targos pushed a commit that referenced this pull request Oct 6, 2025
PR-URL: #60047
Reviewed-By: theanarkh <theratliter@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
aduh95 pushed a commit that referenced this pull request Oct 22, 2025
PR-URL: #60047
Reviewed-By: theanarkh <theratliter@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
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. c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. report Issues and PRs related to process.report.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants