Skip to content
This repository was archived by the owner on Oct 7, 2024. It is now read-only.

Add @lavamoat/allow-scripts#109

Merged
Gudahtt merged 1 commit intomainfrom
add-allow-scripts
Jun 29, 2021
Merged

Add @lavamoat/allow-scripts#109
Gudahtt merged 1 commit intomainfrom
add-allow-scripts

Conversation

@Gudahtt
Copy link
Copy Markdown
Member

@Gudahtt Gudahtt commented Jun 28, 2021

@lavamoat/allow-scripts has been added to ensure only allowed post-install scripts are run. A .yarnrc file has been added to prevent post-install scripts from running automatically.

A basic 'Contributing' section has been added to explain the new setup command. It also includes instructions on setup, linting, and testing.

@Gudahtt Gudahtt requested a review from a team as a code owner June 28, 2021 22:28
rekmarks
rekmarks previously approved these changes Jun 28, 2021
Copy link
Copy Markdown
Member

@rekmarks rekmarks left a comment

Choose a reason for hiding this comment

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

LGTM!

`@lavamoat/allow-scripts` has been added to ensure only allowed post-
install scripts are run. A `.yarnrc` file has been added to prevent
post-install scripts from running automatically.

A basic 'Contributing' section has been added to explain the new
`setup` command. It also includes instructions on setup, linting, and
testing.
@Gudahtt
Copy link
Copy Markdown
Member Author

Gudahtt commented Jun 29, 2021

Rebased to resolve conflict

@Gudahtt Gudahtt merged commit 1631982 into main Jun 29, 2021
@Gudahtt Gudahtt deleted the add-allow-scripts branch June 29, 2021 16:24
@adonesky1 adonesky1 mentioned this pull request Apr 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants