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

Add @lavamoat/allow-scripts #109

Merged
merged 1 commit into from
Jun 29, 2021
Merged

Add @lavamoat/allow-scripts #109

merged 1 commit into from
Jun 29, 2021

Conversation

Gudahtt
Copy link
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
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
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