-
Notifications
You must be signed in to change notification settings - Fork 29.7k
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
tools: fix lint-pr-url message #37304
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Unrelated to this specific PR, but does this lint job something that doesn't run on backport PRs? I think it might result in false positives if it runs on backport PRs.
It does run on backport PRs, and does produce "false positives" warnings. E.G.: https://github.com/nodejs/node/pull/37289/files#diff-71a7964a5c439435680952125e31e9dd4fca3a427982e6db86770fd0fbbfde63R463 I was aware of this when working on this, so I made sure it never produces errors and always completes as green. If people find it too noisy, we can maybe filter out the backport PR somehow. |
PR-URL: nodejs#37304 Reviewed-By: Darshan Sen <raisinten@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
Landed in a803d4a |
PR-URL: #37304 Reviewed-By: Darshan Sen <raisinten@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
PR-URL: #37304 Reviewed-By: Darshan Sen <raisinten@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
actual PR URL
message may be a bit confusing, this PR specifies it's talking about the current PR.