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.

Sign up for GitHub

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

BREAKING CHANGE(fs): stop changing folder permissions #540

Closed
Tracked by #443
wraithgar opened this issue Aug 25, 2022 · 1 comment · Fixed by npm/cli#5710
Closed
Tracked by #443

BREAKING CHANGE(fs): stop changing folder permissions #540

wraithgar opened this issue Aug 25, 2022 · 1 comment · Fixed by npm/cli#5710
Assignees
@nlf
Labels
feature
Milestone
v9.0.0

Comments

@wraithgar
Copy link
Member

Problem statement outlined here: npm/rfcs#546

New behavior here npm/rfcs#546 (comment)

References:

npm/cli#4312
npm/cli#4010
npm/cli#3705
@lukekarrys lukekarrys changed the title [breaking change] stop changing folder permissions BREAKING CHANGE(fs): stop changing folder permissions Sep 2, 2022
@lukekarrys lukekarrys mentioned this issue Sep 2, 2022
Closed
32 tasks
@lukekarrys lukekarrys added this to the v9.0.0 milestone Sep 2, 2022
@nlf
Copy link

nlf commented Oct 13, 2022 •
edited

Removing file ownership changes

nlf added a commit to npm/cli that referenced this issue Oct 18, 2022
@nlf
fix: remove chownr and mkdirp-infer-owner
97d3c26 
closes npm/statusboard#540

BREAKING CHANGE: npm will no longer attempt to modify ownership of files it creates
@nlf nlf mentioned this issue Oct 18, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nlf nlf

Labels
feature
Projects
None yet
Milestone
v9.0.0
Development

Successfully merging a pull request may close this issue.

fix: remove chownr and mkdirp-infer-owner npm/cli
4 participants
@wraithgar @nlf @darcyclarke @lukekarrys