Skip to content

workflows/scheduled: fix excessive-permissions zizmor findings - #195309

Merged
SMillerDev merged 1 commit into
masterfrom
zizmor-scheduled-excessive-permissions
Dec 15, 2024
Merged

workflows/scheduled: fix excessive-permissions zizmor findings#195309
SMillerDev merged 1 commit into
masterfrom
zizmor-scheduled-excessive-permissions

Conversation

@samford

@samford samford commented Dec 15, 2024

Copy link
Copy Markdown
Member

Important: Do not tick a checkbox if you haven鈥檛 performed its action. Honesty is indispensable for a smooth review process.

In the following questions <cask> is the token of the cask you're submitting.

After making any changes to a cask, existing or new, verify:

Additionally, if adding a new cask:

  • Named the cask according to the token reference.
  • Checked the cask was not already refused (add your cask's name to the end of the search field).
  • brew audit --cask --new <cask> worked successfully.
  • HOMEBREW_NO_INSTALL_FROM_API=1 brew install --cask <cask> worked successfully.
  • brew uninstall --cask <cask> worked successfully.

This addresses the excessive-permissions error from zizmor by moving the issues: write permission to the audit_online job where it's needed.

This addresses the `excessive-permissions` error from `zizmor` by
moving the `issues: write` permission to the `audit_online` job where
it's needed.
@SMillerDev
SMillerDev merged commit dafb637 into master Dec 15, 2024
@SMillerDev
SMillerDev deleted the zizmor-scheduled-excessive-permissions branch December 15, 2024 18:35
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.

3 participants