Skip to content

Use allow/deny list in docs - #1699

Closed
luciomartinez wants to merge 55 commits into
npm:release/v7.0.0-betafrom
luciomartinez:allow-deny-list
Closed

Use allow/deny list in docs#1699
luciomartinez wants to merge 55 commits into
npm:release/v7.0.0-betafrom
luciomartinez:allow-deny-list

Conversation

@luciomartinez

@luciomartinezluciomartinez commented Aug 21, 2020

Copy link
Copy Markdown
Contributor

Update documentation to use allowlist and denylist terms.

claudiahdzand others added 30 commits June 24, 2020 16:51
@claudiahdz
chore(docs): fixed links to cli commands
2e05298
This reverts commit 0eac801.
PR-URL: #1459
Credit: @claudiahdzClose: #1459
Reviewed-by: @claudiahdz
@peterfich@claudiahdz
Update npm-link.md
0ca3509
Fix typo.
PR-URL: #1283
Credit: @peterfichClose: #1283
Reviewed-by: @claudiahdz
@maxwellgerber@claudiahdz
docs: Add note about dropped * filenames
3dd429e
In npm/npm-packlist#32, NPM started to drop filenames containing the `*` character.
See:
- #1096
- #1048
PR-URL: #1377
Credit: @maxwellgerberClose: #1377
Reviewed-by: @claudiahdz
@kemitchell@claudiahdz
spdx-license-ids@3.0.5
c49b6ae
PR-URL: #1418
Credit: @kemitchellClose: #1418
Reviewed-by: @claudiahdz
@seanpoulter@claudiahdz
docs: Fix typo
9a2e2e7
PR-URL: #1429
Credit: @seanpoulterClose: #1429
Reviewed-by: @claudiahdz
@vanishcode@claudiahdz
fix: wrong npm doctor command result
b7ad775
@vanishcode@claudiahdz
test: add test for npm doctor that ping registry returns error
1aec4cb
@vanishcode@claudiahdz
style: fix lint error with no trailing comma
479e45c
PR-URL: #1416
Credit: @vanishcodeClose: #1416
Reviewed-by: @claudiahdz
@claudiahdz
chore: remove auth info from logs
a9857b8
@claudiahdz
npm-registry-fetch@4.0.5
94eca63
@claudiahdz
docs: changelog for 6.14.6
66aab41
@claudiahdz
update AUTHORS
46e91d9
@claudiahdz
6.14.6
66092d5
@claudiahdz
libnpx@10.2.4
6a6151f
@claudiahdz
deps: update mkdirp on tacks
2d78481
@claudiahdz
deps: uninstall npm-registry-couchapp
4e129d1
@claudiahdz
deps: fix outdated dev deps
8e1869e
@jasisk@claudiahdz
fix: npm explore spawn shell correctly
de51088
PR-URL: #784
Credit: @jasiskClose: #784
Reviewed-by: @claudiahdz
@claudiahdz
fix: git tag handling regression on shrinkwrap
36e6c01
@bz2@claudiahdz
Fix package id in shrinkwrap lifecycle step output
1961c93
Currently all logging related to shrinkwrap steps reports 'undefined'
for the package in output and log messages.
This is due to the package associated with the `idealTree` being
recreated in the `savePackageJson()` method which precedes these
steps. For now, just copy forward the `_id` attribute which lifecycle
logging expects, but note that mutating `package` here is surprising.
Fixesnpm/npm#20756
PR-URL: #288
Credit: @bz2Close: #288
Reviewed-by: @claudiahdz
Danielle Adams@claudiahdz
fix: gracefully handle error during npm install
8788889
PR-URL: #1009
Credit: @danielleadamsClose: #1009
Reviewed-by: @claudiahdz
@ruyadorno@claudiahdz
fix: npm ls --parseable --long output
6fe2bdc
PR-URL: #1547
Credit: @ruyadornoClose: #1547
Reviewed-by: @claudiahdz
@claudiahdz
bin-links@1.1.8
dc21422
@claudiahdz
gentle-fs@2.3.1
d341f88
@claudiahdz
libcipm@4.0.8
3e168d4
@claudiahdz
npm-audit-report@1.3.3
6ae942a
@claudiahdz
npm-lifecycle@3.1.5
6a35e3d
@claudiahdz
docs: changelog for 6.14.7
e6e8d29
@claudiahdz
update AUTHORS
a81da2b
@claudiahdz
6.14.7
42e4625
@luciomartinez
luciomartinez requested a review from a team as a code ownerAugust 21, 2020 21:06
ruyadorno
ruyadorno suggested changes Aug 27, 2020
View reviewed changes

@ruyadornoruyadorno left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

awesome! thanks for the contribution @luciomartinez ❤️

I find that these phrases need a better construction rather than just replacing the words there but again, thank you so much for bringing this up. We'll make sure to include this in the next release as soon as the requested changes are addressed. 😊

Comment threaddocs/content/configuring-npm/package-json.md Outdated
Comment threaddocs/content/configuring-npm/package-json.md Outdated
Comment threaddocs/content/configuring-npm/package-json.md Outdated
Comment threaddocs/content/using-npm/developers.md Outdated
luciomartinezand others added 4 commits August 28, 2020 17:58
@luciomartinez@ruyadorno
Update phrase
5db2908
Co-authored-by: Ruy Adorno <ruyadorno@github.com>
@luciomartinez@ruyadorno
Update phrase
2f1ce13
Co-authored-by: Ruy Adorno <ruyadorno@github.com>
@luciomartinez@ruyadorno
Update phrase
ba6f2ee
Co-authored-by: Ruy Adorno <ruyadorno@github.com>
@luciomartinez@ruyadorno
Update phrase
75cd049
Co-authored-by: Ruy Adorno <ruyadorno@github.com>
@luciomartinez

luciomartinez commented Aug 28, 2020

Copy link
Copy Markdown
ContributorAuthor

@ruyadorno cool! I was thinking to wrap the terms within code blocks to stand out but this is much better, keeping a more readable grammar. Thanks a lot

@pkrasicki

pkrasicki commented Aug 29, 2020

Copy link
Copy Markdown

I just noticed this PR and the original text is much easier to understand for me. It seems that everyone knows what words whitelist and blacklist mean. Why did you decide to replace them with more words that don't fit so well in this context?

@ljharb

ljharb commented Aug 29, 2020

Copy link
Copy Markdown
Contributor

What they mean tho is “allow list” and “block list”, so this is actually clearer and a much better fit.

@pkrasicki

pkrasicki commented Aug 29, 2020•
edited
Loading

Copy link
Copy Markdown

Perhaps, but compare those two sentences:
You can also blacklist instead of whitelist operating systems - this is the original
You can also block list instead of allow list operating systems - replacement

The second one doesn't make any sense, which is why author of this PR had to add more changes.

@ljharb

ljharb commented Aug 29, 2020

Copy link
Copy Markdown
Contributor

Sure, the verb usage is different, but these are still better terms and worth the additional changes.

@darcyclarkedarcyclarke added the Release 6.x work is associated with a specific npm 6 release label Sep 1, 2020
@darcyclarke
darcyclarke changed the base branch from latest to release/v7.0.0-betaSeptember 18, 2020 18:30
@darcyclarkedarcyclarke added Release 7.x work is associated with a specific npm 7 release semver:patch semver patch level for changes and removed Release 6.x work is associated with a specific npm 6 release labels Sep 18, 2020
ruyadorno pushed a commit that referenced this pull request Sep 22, 2020
Lucio Martinez@ruyadorno
Use allow/deny list in docs
5837a48
PR-URL: #1699
Credit: @luciomartinezClose: #1699
Reviewed-by: @ruyadorno
@ruyadornoruyadorno mentioned this pull request Sep 22, 2020
Merged
@ruyadorno

ruyadorno commented Sep 22, 2020

Copy link
Copy Markdown
Contributor

Landed in 5837a48

@luciomartinez
luciomartinez deleted the allow-deny-list branch September 23, 2020 09:34
@OmrisnykOmrisnyk mentioned this pull request Nov 27, 2023
Open
@OmrisnykOmrisnyk mentioned this pull request Dec 20, 2023
Open
@OmrisnykOmrisnyk mentioned this pull request Feb 12, 2024
Open
@OmrisnykOmrisnyk mentioned this pull request Mar 22, 2024
Open
@OmrisnykOmrisnyk mentioned this pull request May 29, 2024
Open
@OmrisnykOmrisnyk mentioned this pull request Nov 7, 2024
Open
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@ruyadornoruyadornoruyadorno approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Release 7.xwork is associated with a specific npm 7 releasesemver:patchsemver patch level for changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

17 participants

@luciomartinez@pkrasicki@ljharb@ruyadorno@darcyclarke@claudiahdz@peterfich@maxwellgerber@kemitchell@seanpoulter@vanishcode@jasisk@bz2@LocutusOfBorg@sandratatarevicova@cclauss@aduh95