Skip to content

revise adr to support ssh#156

Merged
ericsciple merged 1 commit intomasterfrom
users/ericsciple/m166ssh
Feb 25, 2020
Merged

revise adr to support ssh#156
ericsciple merged 1 commit intomasterfrom
users/ericsciple/m166ssh

Conversation

@ericsciple
Copy link
Copy Markdown
Contributor

@ericsciple ericsciple commented Feb 13, 2020

Proposing adding SSH support. This will help for users scripting submodule checkout.

Will open a separate ADR for a first-class submodule input.

@peaceiris
Copy link
Copy Markdown

Does this proposal mean that we can also do git push with the SSH key? If yes, we can solve this problem: (Github action not triggering gh-pages upon push - GitHub Community Forum) without setting up a personal access token or SSH key by ourself.

Now, the GitHub Pages Actions provides the SSH key input option to avoid the problem. It is useful that the setting up SSH key is supported by actions/checkout.

Thank you.

@ericsciple
Copy link
Copy Markdown
Contributor Author

@peaceiris yes the ssh key will be persisted so you can script authenticated git commands. Will be removed at the end of the job.

@ericsciple ericsciple force-pushed the users/ericsciple/m166ssh branch from 8d97a3b to 2db4320 Compare February 19, 2020 17:55
@ericsciple
Copy link
Copy Markdown
Contributor Author

reviewed offline w/ @thechrisjohnson

@ericsciple ericsciple merged commit 096e927 into master Feb 25, 2020
@ericsciple ericsciple deleted the users/ericsciple/m166ssh branch February 25, 2020 15:00
@ericsciple ericsciple mentioned this pull request Feb 25, 2020
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.

3 participants