X Tutup
The Wayback Machine - https://web.archive.org/web/20220323011126/https://github.com/nodejs/node/pull/35254
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

Jump to bottom

doc: edit subpath export patterns introduction #35254

Merged
merged 1 commit into from Sep 21, 2020
Merged

doc: edit subpath export patterns introduction #35254

merged 1 commit into from Sep 21, 2020

Conversation

Trott
Copy link
Member

@Trott Trott commented Sep 18, 2020

  • Use parallel construction in the two sentences
  • Backticks around package.json to match rest of file
  • Add comma for readability
  • Own the recommendation ("we recommend")
Checklist

@nodejs-github-bot
Copy link
Contributor

@nodejs-github-bot nodejs-github-bot commented Sep 18, 2020

Review requested:

@nodejs-github-bot nodejs-github-bot added the doc label Sep 18, 2020
jkrems
jkrems approved these changes Sep 18, 2020
View changes
mhdawson
mhdawson reviewed Sep 18, 2020
View changes
doc/api/packages.md
subpaths this can start to cause package.json bloat and maintenance issues.
For packages with a small number of exports, we recommend explicitly listing
each exports subpath entry. But for packages that have large numbers of
subpaths, this might cause `package.json` bloat and maintenance issues.
Copy link
Member

@mhdawson mhdawson Sep 18, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
subpaths, this might cause `package.json` bloat and maintenance issues.
this might cause `package.json` bloat and cause maintenance issues.

mhdawson
mhdawson approved these changes Sep 18, 2020
View changes
Copy link
Member

@mhdawson mhdawson left a comment

LGTM

guybedford
guybedford approved these changes Sep 18, 2020
View changes
Copy link
Contributor

@guybedford guybedford left a comment

Thanks for taking a look at this.

@Trott
doc: edit subpath export patterns introduction
98768f3 
* Use parallel construction in the two sentences
* Backticks around _package.json_ to match rest of file
* Add comma for readability
* Own the recommendation ("we recommend")

PR-URL: nodejs#35254
Reviewed-By: Jan Krems <jan.krems@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Guy Bedford <guybedford@gmail.com>
@Trott Trott merged commit 98768f3 into nodejs:master Sep 21, 2020
8 checks passed
@Trott
Copy link
Member Author

@Trott Trott commented Sep 21, 2020

Landed in 98768f3

@Trott Trott deleted the packages branch Sep 21, 2020
@ruyadorno ruyadorno added the dont-land-on-v14.x label Sep 21, 2020
@ruyadorno
Copy link
Member

@ruyadorno ruyadorno commented Sep 21, 2020

The doc/api/packages.md refactor has not yet landed on v14.x:

Until that's all resolved we should not land this patch on v14.x

@MylesBorins MylesBorins removed the dont-land-on-v14.x label Sep 24, 2020
MylesBorins added a commit that referenced this issue Sep 24, 2020
@Trott @MylesBorins
doc: edit subpath export patterns introduction
d7282c0 
* Use parallel construction in the two sentences
* Backticks around _package.json_ to match rest of file
* Add comma for readability
* Own the recommendation ("we recommend")

PR-URL: #35254
Reviewed-By: Jan Krems <jan.krems@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Guy Bedford <guybedford@gmail.com>
@MylesBorins MylesBorins mentioned this pull request Sep 29, 2020
Merged
@aduh95 aduh95 mentioned this pull request Oct 23, 2020
Closed
3 tasks
aduh95 added a commit to aduh95/node that referenced this issue Oct 23, 2020
@Trott @aduh95
doc: edit subpath export patterns introduction
55d15e6 
* Use parallel construction in the two sentences
* Backticks around _package.json_ to match rest of file
* Add comma for readability
* Own the recommendation ("we recommend")

PR-URL: nodejs#35254
Reviewed-By: Jan Krems <jan.krems@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Guy Bedford <guybedford@gmail.com>
MylesBorins added a commit that referenced this issue Nov 3, 2020
@Trott @MylesBorins
doc: edit subpath export patterns introduction
e292b02 
* Use parallel construction in the two sentences
* Backticks around _package.json_ to match rest of file
* Add comma for readability
* Own the recommendation ("we recommend")

Backport-PR-URL: #35757
PR-URL: #35254
Reviewed-By: Jan Krems <jan.krems@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Guy Bedford <guybedford@gmail.com>
@aduh95 aduh95 added the backported-to-v12.x label Nov 3, 2020
@MylesBorins MylesBorins mentioned this pull request Nov 3, 2020
Merged
MylesBorins added a commit that referenced this issue Nov 16, 2020
@Trott @MylesBorins
doc: edit subpath export patterns introduction
5f0b157 
* Use parallel construction in the two sentences
* Backticks around _package.json_ to match rest of file
* Add comma for readability
* Own the recommendation ("we recommend")

Backport-PR-URL: #35757
PR-URL: #35254
Reviewed-By: Jan Krems <jan.krems@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Guy Bedford <guybedford@gmail.com>
joesepi added a commit to joesepi/node that referenced this issue Jan 8, 2021
@Trott @joesepi
doc: edit subpath export patterns introduction
4ae4ef4 
* Use parallel construction in the two sentences
* Backticks around _package.json_ to match rest of file
* Add comma for readability
* Own the recommendation ("we recommend")

PR-URL: nodejs#35254
Reviewed-By: Jan Krems <jan.krems@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Guy Bedford <guybedford@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jkrems jkrems

@guybedford guybedford

@mhdawson mhdawson

Assignees
No one assigned
Labels
backported-to-v12.x doc
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet
8 participants
@Trott @nodejs-github-bot @ruyadorno @jkrems @guybedford @mhdawson @MylesBorins @aduh95
X Tutup