Skip to content

ci: declare contents: read on validate_pyinstaller workflow#8992

Merged
roger-zhangg merged 2 commits into
aws:developfrom
arpitjain099:ci/add-permissions
May 13, 2026
Merged

ci: declare contents: read on validate_pyinstaller workflow#8992
roger-zhangg merged 2 commits into
aws:developfrom
arpitjain099:ci/add-permissions

Conversation

@arpitjain099
Copy link
Copy Markdown
Contributor

The validate_pyinstaller.yml workflow currently leaves GITHUB_TOKEN scope implicit. It does PR-time PyInstaller builds across Linux / macOS / Windows runners — no commits, releases, or comments. contents: read is the right floor for the checkout step.

PR-triggered build of the PyInstaller bundle for linux/macos/windows.
Each job is a build matrix that runs docker + python + pyinstaller. No
push, no comment, no release write - contents: read suffices.

Signed-off-by: Arpit Jain <arpitjain099@gmail.com>
@arpitjain099 arpitjain099 requested a review from a team as a code owner May 13, 2026 08:38
@github-actions github-actions Bot added pr/external stage/needs-triage Automatically applied to new issues and PRs, indicating they haven't been looked at. labels May 13, 2026
Copy link
Copy Markdown
Collaborator

@aws-sam-cli-bot aws-sam-cli-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

redacted

@roger-zhangg roger-zhangg added this pull request to the merge queue May 13, 2026
Merged via the queue into aws:develop with commit 4113a20 May 13, 2026
53 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr/external stage/needs-triage Automatically applied to new issues and PRs, indicating they haven't been looked at.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants