Skip to content
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

Closed
wants to merge 1 commit into from

Conversation

jasnell
Copy link
Member

@jasnell jasnell commented Apr 28, 2021

Fixes: #17669
Signed-off-by: James M Snell jasnell@gmail.com

Fixes: nodejs#17669
Signed-off-by: James M Snell <jasnell@gmail.com>
@github-actions github-actions bot added doc Issues and PRs related to the documentations. repl Issues and PRs related to the REPL subsystem. labels Apr 28, 2021
@jasnell jasnell added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Apr 28, 2021
@benjamingr
Copy link
Member

benjamingr commented Apr 28, 2021

@jasnell

BTW, why do your PRs all have:

Signed-off-by: James M Snell jasnell@gmail.com

Is that a byproduct of using some particular software to author commits or does that signify anything?

@jasnell
Copy link
Member Author

jasnell commented Apr 28, 2021

Is that a byproduct of using some particular software to author commits or does that signify anything?

It's added by using git commit --signoff

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:

The DCO is a legally binding statement that asserts that you are the creator of your contribution, and that you license the work under the Apache License Version 2.0.

To indicate that you agree to the terms of the DCO, you “sign off” your contribution by adding a line with your name and e-mail address to every Git commit message:

Signed-off-by: Jane Doe jane.doe@example.com

You must use your real name (no pseudonyms or anonymous contributions are allowed.)

Your signature certifies that you are either the author of the contribution or have the right to submit it under the open-source license used by the DITA Open Toolkit project.

If you set your user.name and user.email as part of your Git configuration, you can sign your commit automatically with git commit -s.

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 ;-)

@jasnell jasnell added the fast-track PRs that do not need to wait for 48 hours to land. label Apr 28, 2021
@jasnell
Copy link
Member Author

jasnell commented Apr 28, 2021

This shouldn't need to wait the full 48 hours to land. Please 👍🏻 to fast-track

@jasnell
Copy link
Member Author

jasnell commented Apr 28, 2021

Landed in 346f0e9

@jasnell jasnell closed this Apr 28, 2021
jasnell added a commit that referenced this pull request Apr 28, 2021
Fixes: #17669
Signed-off-by: James M Snell <jasnell@gmail.com>

PR-URL: #38449
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
targos pushed a commit that referenced this pull request Apr 29, 2021
Fixes: #17669
Signed-off-by: James M Snell <jasnell@gmail.com>

PR-URL: #38449
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
@targos targos mentioned this pull request May 3, 2021
targos pushed a commit that referenced this pull request May 30, 2021
Fixes: #17669
Signed-off-by: James M Snell <jasnell@gmail.com>

PR-URL: #38449
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
targos pushed a commit that referenced this pull request Jun 5, 2021
Fixes: #17669
Signed-off-by: James M Snell <jasnell@gmail.com>

PR-URL: #38449
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
targos pushed a commit that referenced this pull request Jun 5, 2021
Fixes: #17669
Signed-off-by: James M Snell <jasnell@gmail.com>

PR-URL: #38449
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
targos pushed a commit that referenced this pull request Jun 11, 2021
Fixes: #17669
Signed-off-by: James M Snell <jasnell@gmail.com>

PR-URL: #38449
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@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. doc Issues and PRs related to the documentations. fast-track PRs that do not need to wait for 48 hours to land. repl Issues and PRs related to the REPL subsystem.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

REPL Await invalidates const
5 participants