Skip to content

Adds a note about workspaces in the enableScripts documentation#4781

Merged
arcanis merged 1 commit intomasterfrom
mael/enable-scripts-workspaces
Aug 19, 2022
Merged

Adds a note about workspaces in the enableScripts documentation#4781
arcanis merged 1 commit intomasterfrom
mael/enable-scripts-workspaces

Conversation

@arcanis
Copy link
Copy Markdown
Member

@arcanis arcanis commented Aug 18, 2022

What's the problem this PR addresses?

The enableScripts setting currently only applies to third-party packages. I believe it's because we ideally would want installs to be made with enableScripts set to false by default (for security reasons), but there's no point into blocking the workspaces' postinstall scripts by default (since it doesn't improve security).

Closes #4780
Ref #4386
Ref #4547

How did you fix it?

Updated the documentation.

Checklist

  • I have set the packages that need to be released for my changes to be effective.
  • I will check that all automated PR checks pass before the PR gets reviewed.

@arcanis arcanis merged commit 6687804 into master Aug 19, 2022
@arcanis arcanis deleted the mael/enable-scripts-workspaces branch August 19, 2022 08:18
merceyz pushed a commit that referenced this pull request Aug 19, 2022
merceyz pushed a commit that referenced this pull request Sep 21, 2022
merceyz pushed a commit that referenced this pull request Oct 5, 2022
merceyz pushed a commit that referenced this pull request Nov 16, 2022
merceyz pushed a commit that referenced this pull request Dec 20, 2022
merceyz pushed a commit that referenced this pull request Feb 1, 2023
merceyz pushed a commit that referenced this pull request Mar 16, 2023
krassowski added a commit to krassowski/jupyterlab that referenced this pull request Mar 24, 2023
merceyz pushed a commit that referenced this pull request May 1, 2023
merceyz pushed a commit that referenced this pull request Jun 1, 2023
merceyz pushed a commit that referenced this pull request Oct 21, 2023
)

Update yarnrc.json

(cherry picked from commit 6687804)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug?]: disabled enableScripts setting still triggers postinstall script

1 participant