Skip to content

ci: harden workflows#232

Merged
lydell merged 1 commit intoprettier:mainfrom
Fdawgs:ci/actions
Jan 1, 2023
Merged

ci: harden workflows#232
lydell merged 1 commit intoprettier:mainfrom
Fdawgs:ci/actions

Conversation

@Fdawgs
Copy link
Copy Markdown
Contributor

@Fdawgs Fdawgs commented Nov 8, 2022

This PR:

  • Removes Git credentials/SSH keys after checkout as a security precaution by setting persist-credentials to false, they are not used after the initial checkout
  • Declares the minimum permissions for CI workflows to run at the workflow level, following principle of least privilege; see related GitHub security post

@lydell lydell merged commit 1218d22 into prettier:main Jan 1, 2023
@lydell
Copy link
Copy Markdown
Member

lydell commented Jan 1, 2023

Let’s try this. Thanks!

@Fdawgs Fdawgs deleted the ci/actions branch January 1, 2023 11:13
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.

2 participants