This is an archive of the discontinued LLVM Phabricator instance.

feat: harden permissions for all github workflows
ClosedPublic

Authored by joycebrum on Feb 15 2023, 10:16 AM.

Details

Summary

Signed-off-by: Joyce Brum <joycebrum@google.com>

Diff Detail

Event Timeline

joycebrum created this revision.Feb 15 2023, 10:16 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 15 2023, 10:16 AM
Herald added a subscriber: jvesely. · View Herald Transcript
joycebrum requested review of this revision.Feb 15 2023, 10:16 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 15 2023, 10:16 AM

Hi, let me know if any permission is missing, I tried to look into the actions to figured out but may have missed something.

tstellar accepted this revision.Feb 22 2023, 1:41 PM

LGTM. I tested this here with our stable branch CI jobs: https://github.com/llvm/llvm-project-release-prs/pull/309

This revision is now accepted and ready to land.Feb 22 2023, 1:41 PM

@joycebrum Do you need me to commit this for you?

@joycebrum Do you need me to commit this for you?

Please! I think I don't have access to do that. Thanks!

This revision was automatically updated to reflect the committed changes.