-
Notifications
You must be signed in to change notification settings - Fork 29.6k
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
repl: document top level await limitation with const/let #38449
Conversation
Fixes: nodejs#17669 Signed-off-by: James M Snell <[email protected]>
BTW, why do your PRs all have:
Is that a byproduct of using some particular software to author commits or does that signify anything? |
It's added by using As for what it signifies... The answer is we technically should be requiring it for all commits from everyone as part of the DCO (Developer Certificate of Origin) requirements. This from the Dita project provides the most succinct explanation I've been able to find:
For Node.js we do not enforce the "real name" requirement, but there's been a long-standing open discussion in the TSC pending feedback from the Linux Foundation legal representatives about whether we need to actually enforce the requirement of having the sign-off. I add it myself in a subtle attempt at trying to make it catch on ;-) |
This shouldn't need to wait the full 48 hours to land. Please 👍🏻 to fast-track |
Landed in 346f0e9 |
Fixes: #17669 Signed-off-by: James M Snell <[email protected]> PR-URL: #38449 Reviewed-By: Jiawen Geng <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Yongsheng Zhang <[email protected]>
Fixes: #17669 Signed-off-by: James M Snell <[email protected]> PR-URL: #38449 Reviewed-By: Jiawen Geng <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Yongsheng Zhang <[email protected]>
Fixes: #17669 Signed-off-by: James M Snell <[email protected]> PR-URL: #38449 Reviewed-By: Jiawen Geng <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Yongsheng Zhang <[email protected]>
Fixes: #17669 Signed-off-by: James M Snell <[email protected]> PR-URL: #38449 Reviewed-By: Jiawen Geng <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Yongsheng Zhang <[email protected]>
Fixes: #17669 Signed-off-by: James M Snell <[email protected]> PR-URL: #38449 Reviewed-By: Jiawen Geng <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Yongsheng Zhang <[email protected]>
Fixes: #17669 Signed-off-by: James M Snell <[email protected]> PR-URL: #38449 Reviewed-By: Jiawen Geng <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Yongsheng Zhang <[email protected]>
Fixes: #17669
Signed-off-by: James M Snell [email protected]