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

Add @lavamoat/allow-scripts #47

Merged
merged 1 commit into from
Sep 22, 2021
Merged

Add @lavamoat/allow-scripts #47

merged 1 commit into from
Sep 22, 2021

Conversation

Gudahtt
Copy link
Member

@Gudahtt Gudahtt commented Sep 16, 2021

We now have an allowlist for all post-install scripts. The standard setup script has been added, along with new contributor documentation in the README to explain this script.

@Gudahtt Gudahtt requested a review from a team as a code owner September 16, 2021 19:24
@Gudahtt
Copy link
Member Author

Gudahtt commented Sep 16, 2021

This depends upon #46

@Gudahtt Gudahtt marked this pull request as draft September 16, 2021 19:29
@Gudahtt Gudahtt force-pushed the migrate-to-github-actions branch 2 times, most recently from bb89679 to a7469ec Compare September 16, 2021 20:04
@Gudahtt Gudahtt force-pushed the migrate-to-github-actions branch from a7469ec to f267347 Compare September 16, 2021 20:31
Base automatically changed from migrate-to-github-actions to main September 21, 2021 23:25
@Gudahtt Gudahtt marked this pull request as ready for review September 21, 2021 23:26
We now have an allowlist for all post-install scripts. The standard
`setup` script has been added, along with new contributor documentation
in the README to explain this script.
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!

@Gudahtt Gudahtt merged commit 6d9c1dd into main Sep 22, 2021
@Gudahtt Gudahtt deleted the add-allow-scripts branch September 22, 2021 00:00
@adonesky1 adonesky1 mentioned this pull request Apr 14, 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