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

auto-pr: drop HOMEBREW_NO_AUTO_UPDATE #65

Merged
merged 1 commit into from
Nov 1, 2023
Merged

Conversation

woodruffw
Copy link
Member

This may be causing our PAT pains.

This _may_ be causing our PAT pains.

Signed-off-by: William Woodruff <[email protected]>
@woodruffw
Copy link
Member Author

@woodruffw woodruffw marked this pull request as ready for review November 1, 2023 16:14
@woodruffw
Copy link
Member Author

Looks like this did the trick. For retrospection: the problem here was (?) that our local branches weren't being made against the latest upstream/master, so they'd implicitly contain workflow changes (which would then cause the PAT to fail, since it correctly doesn't include the workflow scope).

I'm pretty sure this is the right fix, and it appears to be working. But we'll see.

@woodruffw woodruffw merged commit 0ad1b4b into main Nov 1, 2023
1 check passed
@woodruffw woodruffw deleted the ww/drop-no-auto-update branch November 1, 2023 16:17
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants