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 preview release process to comment on PRs #33

Conversation

rtyley
Copy link
Member

@rtyley rtyley commented Jan 5, 2024

guardian/gha-scala-library-release-workflow#19 introduces support for publishing preview releases on PR branches, but note this does use the pull-requests: write permission in order to comment on the PR, to announce that the release has been published.

Consequently, we need to additionally grant the permission.

guardian/gha-scala-library-release-workflow#19
introduces support for publishing preview releases, but note this does
use the `pull-requests: write` permission in order to comment on the PR
to announce that the release has been published - so we need to
additionally grant that permission:

https://github.com/guardian/gha-scala-library-release-workflow/pull/19/files#r1441589383
@rtyley rtyley merged commit b8451c8 into main Jan 5, 2024
2 checks passed
@rtyley rtyley deleted the allow-release-process-to-comment-on-prs-to-announce-preview-releases branch January 5, 2024 17:02
@guardian guardian deleted a comment from github-actions bot Jan 5, 2024
rtyley added a commit to guardian/play-googleauth that referenced this pull request Jan 5, 2024
rtyley added a commit to guardian/marley that referenced this pull request Jan 5, 2024
rtyley added a commit to guardian/facia-scala-client that referenced this pull request Jan 5, 2024
rtyley added a commit to guardian/play-secret-rotation that referenced this pull request Jan 5, 2024
rtyley added a commit to guardian/redirect-resolver that referenced this pull request Jan 5, 2024
rtyley added a commit to guardian/targeting-client that referenced this pull request Jan 5, 2024
@rtyley
Copy link
Member Author

rtyley commented Jan 5, 2024

Incidentally, to update the 6 or so other repos currently using to make this change, I changed directory into each repo in turn on my local box and executed:

git reset --hard && git checkout main && git pull && cp ../etag-caching/.github/workflows/release.yml .github/workflows/release.yml && git diff && git add .github/workflows/release.yml && git commit -m "Update gha-scala-library-release-workflow params - see https://github.com/guardian/etag-caching/pull/33" && git push

...the script is a little aggressive because it throws away whatever's in your working directory, it should probably clone fresh copies of the repo or something...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant