Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

Updates to ssh key enablement for Cloud projects #8416

Merged

Conversation

meker12
Copy link
Contributor

@meker12 meker12 commented Dec 18, 2020

Purpose of this pull request

Add additional method to add and update SSH keys. Merges changes introduced in #8379.

Affected DevDocs pages

* Added an additional method regarding adding a SSH key

* Apply suggestions from code review

Co-authored-by: Barny Shergold <[email protected]>
@meker12 meker12 added 2.x Technical Updates to the code or processes that alter the technical content of the doc labels Dec 18, 2020
@meker12 meker12 self-assigned this Dec 18, 2020
Copy link
Contributor

@hguthrie hguthrie left a comment

Choose a reason for hiding this comment

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

Are you still updating this? Too much use of the word "will" and needs work to be consistent with Cloud guide tone and formatting.

@meker12
Copy link
Contributor Author

meker12 commented Dec 21, 2020

Are you still updating this? Too much use of the word "will" and needs work to be consistent with Cloud guide tone and formatting.

Yes-- I merged the other PR to the SSH enablement topic. Just working on merging this into integration branch to fix it up.

@meker12 meker12 changed the base branch from master to mae-ssh-enablement-improvements December 21, 2020 17:58
- Replaced test SSH connection through GitHub instructions
with link to GitHub documentation
- Editorial revision and clean up
@meker12 meker12 requested a review from hguthrie December 23, 2020 18:49
@@ -38,23 +38,9 @@ If the directory does not exist or has no SSH key files, you must generate at le

If you have at least one SSH key in your directory, add the key to your Magento and GitHub accounts:

- [Add an SSH key to your GitHub account](https://help.github.com/articles/adding-a-new-ssh-key-to-your-github-account/)
- [Add an SSH key to your GitHub account](https://help.github.com/articles/adding-a-new-ssh-key-to-your-github-account/) and [test the SSH connection](https://docs.github.com/en/free-pro-team@latest/github/authenticating-to-github/testing-your-ssh-connection).
Copy link
Contributor

Choose a reason for hiding this comment

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

😄

@meker12 meker12 merged commit edbac0f into mae-ssh-enablement-improvements Jan 4, 2021
@ghost
Copy link

ghost commented Jan 4, 2021

Hi @meker12, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

meker12 added a commit that referenced this pull request Jan 4, 2021
* Updates based on review feedback

- Replaced instructions for testing ssh connection
with link to GitHub documentation
- Editorial cleanup

* Updates to ssh key enablement for Cloud projects (#8416)

* Added an additional method regarding adding a SSH key (#8379)

* Added an additional method regarding adding a SSH key

* Apply suggestions from code review

Co-authored-by: Barny Shergold <[email protected]>

* Updates based on review feedback

- Replaced test SSH connection through GitHub instructions
with link to GitHub documentation
- Editorial revision and clean up

Co-authored-by: Sathiya Prakash <[email protected]>
Co-authored-by: Barny Shergold <[email protected]>

Co-authored-by: Sathiya Prakash <[email protected]>
Co-authored-by: Barny Shergold <[email protected]>
@dshevtsov dshevtsov deleted the mae-updates-to-ssh-key-enablement branch January 19, 2021 20:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2.x Progress: done Technical Updates to the code or processes that alter the technical content of the doc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants