-
Notifications
You must be signed in to change notification settings - Fork 29.8k
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
doc: document self-referencing a package name #31680
Conversation
d206dcb
to
fb077ef
Compare
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.
Thanks for adding this! Looks good, one thing that may be confusing.
/cc @nodejs/modules-active-members |
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
6d38758
to
256fe12
Compare
Not sure about the protocol, and there's nothing in the guidelines about this: should I squash the commits? |
Yes, I think this PR would be appropriate for squashing. The commit message should follow the guidelines, such as starting with |
b942e48
to
0873607
Compare
Rebased and squashed. |
Added a section for "Self-referencing a package using its name" that documents importing a package's own exports (this was missed when adding the feature).
302d8d6
to
b260d06
Compare
Sorry for losing track of this. I assume this is good to land. I plan to do it at some point this week if nobody else beats me to it. |
Landed in 9dbe6ab |
Added a section for "Self-referencing a package using its name" that documents importing a package's own exports (this was missed when adding the feature). PR-URL: #31680 Reviewed-By: Jan Krems <[email protected]> Reviewed-By: Myles Borins <[email protected]> Reviewed-By: Geoffrey Booth <[email protected]>
Added a section for "Self-referencing a package using its name" that documents importing a package's own exports (this was missed when adding the feature). PR-URL: #31680 Reviewed-By: Jan Krems <[email protected]> Reviewed-By: Myles Borins <[email protected]> Reviewed-By: Geoffrey Booth <[email protected]>
Added a section for "Self-referencing a package using its name" that documents importing a package's own exports (this was missed when adding the feature). PR-URL: #31680 Reviewed-By: Jan Krems <[email protected]> Reviewed-By: Myles Borins <[email protected]> Reviewed-By: Geoffrey Booth <[email protected]>
Added a section for "Self-referencing a package using its name" that documents importing a package's own exports (this was missed when adding the feature). PR-URL: #31680 Reviewed-By: Jan Krems <[email protected]> Reviewed-By: Myles Borins <[email protected]> Reviewed-By: Geoffrey Booth <[email protected]>
Added a section for "Self-referencing a package using its name" that documents importing a package's own exports (this was missed when adding the feature). PR-URL: #31680 Reviewed-By: Jan Krems <[email protected]> Reviewed-By: Myles Borins <[email protected]> Reviewed-By: Geoffrey Booth <[email protected]>
Added a section for "Self-referencing a package using its name" that documents
importing a package's own exports (this was missed when adding the feature).
Checklist