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

allow workflow content write #19870

Merged
merged 1 commit into from
Feb 12, 2025

Conversation

jmartin-tech
Copy link
Contributor

To enabled branch and commit content must be added

Verification

This can be tested in a fork using a manual trigger when "Settings -> Actions -> Workflow Permissions" is set to "Read repository contents and packages permissions".

To enabled branch and commit `content` must be added
@h00die h00die self-requested a review February 11, 2025 11:01
@h00die
Copy link
Contributor

h00die commented Feb 11, 2025

I'll take a look at this tonight. Thanks for the help @jmartin-tech

@h00die
Copy link
Contributor

h00die commented Feb 12, 2025

Adjusted permissions in my repo per convo on slack.
made the action change to match this PR.
https://github.com/h00die/metasploit-framework/actions/runs/13284563895/job/37090221071
success.

@adfoster-r7 adfoster-r7 added the rn-no-release-notes no release notes label Feb 12, 2025
@h00die h00die merged commit a5d7dfb into rapid7:master Feb 12, 2025
75 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rn-no-release-notes no release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants