Skip to content

stubsabot: use an ssh-key when checking out typeshed #8844

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

Merged
merged 2 commits into from
Oct 4, 2022

Conversation

AlexWaygood
Copy link
Member

Fixes #8434.

This uses a new ssh key that @JelleZijlstra just set up for our repo (this option in Peter Evans's guide to getting checks to run on automated PRs. It seems to be a good option in terms of both security and usability. It's also the option used by oddbird/MetaDeploy here, and they seem pretty happy about it (they wrote a nice blog post about their automated PRs for uprading dependencies here.)

Fixes #8434.

This uses a new ssh key that @JelleZijlstra just set up for our repo ([this option](https://github.com/peter-evans/create-pull-request/blob/main/docs/concepts-guidelines.md#push-using-ssh-deploy-keys) in Peter Evans's guide to getting checks to run on automated PRs. It seems to be a good option in terms of both security and usability. It's also the option used by `oddbird/MetaDeploy` [here](https://github.com/oddbird/MetaDeploy/blob/main/.github/workflows/upgrade-deps.yml#L20), and they seem pretty happy about it (they wrote a nice blog post about their automated PRs for uprading dependencies [here](https://www.oddbird.net/2022/06/01/dependabot-single-pull-request/).)
Copy link
Collaborator

@hauntsaninja hauntsaninja left a comment

Choose a reason for hiding this comment

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

Okey dokey

@hauntsaninja hauntsaninja merged commit 5cb7ef1 into master Oct 4, 2022
@hauntsaninja hauntsaninja deleted the AlexWaygood-stubsabot-pr-checks branch October 4, 2022 21:13
@hauntsaninja
Copy link
Collaborator

Seems like this didn't work

@AlexWaygood
Copy link
Member Author

Yeah.

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.

Checks don't run on stubsabot PRs
3 participants