-
-
Notifications
You must be signed in to change notification settings - Fork 34.2k
win,tools: Add description to signature #59877
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
win,tools: Add description to signature #59877
Conversation
When signing files for Windows, add "Node.js" as the description.
77cd3b9 to
2c0bf67
Compare
|
Test failures look like flaky tests on macOS. Can someone re-run and merge please? |
|
Landed in 85b0e06 |
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>
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>
|
@martincostello this requires manual backport to v20 |
|
I don't know what that involves. |
|
It involves cherry-pick'ing 85b0e06 on a branch targeting the v20.x-staging branch |
|
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 |
|
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. |

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.