Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[workflows] Add top-level permissions for libcxx-build-and-test.yaml #73689

Merged
merged 1 commit into from
Nov 29, 2023

Conversation

tstellar
Copy link
Collaborator

This is the standard convention for our workflow files.

This is the standard convention for our workflow files.
@llvmbot
Copy link
Collaborator

llvmbot commented Nov 28, 2023

@llvm/pr-subscribers-github-workflow

Author: Tom Stellard (tstellar)

Changes

This is the standard convention for our workflow files.


Full diff: https://github.com/llvm/llvm-project/pull/73689.diff

1 Files Affected:

  • (modified) .github/workflows/libcxx-build-and-test.yaml (+3)
diff --git a/.github/workflows/libcxx-build-and-test.yaml b/.github/workflows/libcxx-build-and-test.yaml
index dccfe54d4646767..e929c02d11c1d93 100644
--- a/.github/workflows/libcxx-build-and-test.yaml
+++ b/.github/workflows/libcxx-build-and-test.yaml
@@ -23,6 +23,9 @@ on:
       - 'cmake/**'
       - '.github/workflows/libcxx-build-and-test.yaml'
 
+permissions:
+  contents: read # Default everything to read-only
+
 concurrency:
   group: ${{ github.workflow }}-${{ github.event.pull_request.number }}
   cancel-in-progress: true

@tstellar tstellar merged commit f150ecc into llvm:main Nov 29, 2023
37 of 39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants