Skip to content

Conversation

@martincostello
Copy link
Contributor

When signing files for Windows, add "Node.js" as the description.

This should cause the string highlighted in the UAC elevation prompt screenshot below to be replaced with the string specified as the description.

image

Copilot AI review requested due to automatic review settings September 13, 2025 09:43
@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory. windows Issues and PRs related to the Windows platform. labels Sep 13, 2025

This comment was marked as spam.

When signing files for Windows, add "Node.js" as the description.
@lpinca lpinca added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 27, 2025
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 27, 2025
@nodejs-github-bot
Copy link
Collaborator

@martincostello
Copy link
Contributor Author

Test failures look like flaky tests on macOS.

Can someone re-run and merge please?

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@richardlau richardlau added commit-queue Add this label to land a pull request using GitHub Actions. lts-watch-v20.x PRs that may need to be released in v20.x lts-watch-v22.x PRs that may need to be released in v22.x labels Oct 3, 2025
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 3, 2025
@nodejs-github-bot nodejs-github-bot merged commit 85b0e06 into nodejs:main Oct 3, 2025
65 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in 85b0e06

@martincostello martincostello deleted the sign-tool-description branch October 3, 2025 23:23
targos pushed a commit that referenced this pull request Oct 6, 2025
When signing files for Windows, add "Node.js" as the description.

PR-URL: #59877
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com>
Reviewed-By: Stefan Stojanovic <stefan.stojanovic@janeasystems.com>
aduh95 pushed a commit that referenced this pull request Oct 11, 2025
When signing files for Windows, add "Node.js" as the description.

PR-URL: #59877
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com>
Reviewed-By: Stefan Stojanovic <stefan.stojanovic@janeasystems.com>
@aduh95 aduh95 added backported-to-v22.x PRs backported to the v22.x-staging branch. and removed lts-watch-v22.x PRs that may need to be released in v22.x labels Oct 11, 2025
@bigHt800
Copy link

👀

When signing files for Windows, add "Node.js" as the description.

This should cause the string highlighted in the UAC elevation prompt screenshot below to be replaced with the string specified as the description.

image

@marco-ippolito
Copy link
Member

@martincostello this requires manual backport to v20

@marco-ippolito marco-ippolito added the backport-requested-v20.x PRs awaiting manual backport to the v20.x-staging branch. label Nov 19, 2025
@martincostello
Copy link
Contributor Author

I don't know what that involves.

@marco-ippolito
Copy link
Member

It involves cherry-pick'ing 85b0e06 on a branch targeting the v20.x-staging branch

@martincostello
Copy link
Contributor Author

I won't have access to a computer for several days, so if it's urgent someone else will need to do that.

It sounds like a task that should be automated in-repo that the maintainers can self-manage.

@marco-ippolito
Copy link
Member

I won't have access to a computer for several days, so if it's urgent someone else will need to do that.

It sounds like a task that should be automated in-repo that the maintainers can self-manage.

The commit needs to be adapted because the branches are divergent. If you are interested in making sure this change arrives in v20 feel free to open a backport PR

@aduh95
Copy link
Contributor

aduh95 commented Nov 19, 2025

To clarify, when Marco says "requires", he means "if we want this change on 20.x, it can't happen without a manual backport PR". He pinged you because as the author of the change, you're probably in the best position to do the backport, but it doesn't have to be you, like any work in Node.js someone needs to volunteer to do the work.
Looking at the change, because #58502 was never backported, I don't think this change can be backported anyway, because we don't use signtool, so no action is needed here IIUC.

@aduh95 aduh95 added dont-land-on-v20.x PRs that should not land on the v20.x-staging branch and should not be released in v20.x. and removed backport-requested-v20.x PRs awaiting manual backport to the v20.x-staging branch. labels Nov 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backported-to-v22.x PRs backported to the v22.x-staging branch. dont-land-on-v20.x PRs that should not land on the v20.x-staging branch and should not be released in v20.x. lts-watch-v20.x PRs that may need to be released in v20.x needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory. windows Issues and PRs related to the Windows platform.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants