-
Notifications
You must be signed in to change notification settings - Fork 71
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
Update SSH access command #163
Conversation
Add shorthand command for `ssh -p 2222 vagrant@localhost` `vagrant ssh` works
Thanks for this @DonRichards! Can you make this consistent with what you did here and also address this comment? |
Should probably remove:
As the Port will change to a random port if you already have one vagrant instance running that exposes the Guest 22 on the host as 2222. |
@DonRichards bump. |
@nigelgbanks I would suggest leaving only so others with more complicated environments know this is an option. But it could be a side note instead of a key point |
@@ -65,6 +65,7 @@ Tomcat Manager: | |||
* password: islandora | |||
|
|||
You can connect to the machine via ssh: `ssh -p 2222 vagrant@localhost` | |||
Or shorthand `vagrant ssh` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DonRichards can you change this into two bullets?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DonRichards bump.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was there anything else that was needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At this moment, no. Before I'll merge this, I'd like to have it formatted correctly, and consistent with this entire section. So, if you can do that, it would be great.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DonRichards you got this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Formatted correctly? The update name description or the actual code?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DonRichards I'm specifically referring to the bullet points. Right now you have:
I'm asking that you make it consistent with the rest of the section. Like this:
You can connect to the machine via ssh:
ssh -p 2222 vagrant@localhost
vagrant ssh
Does that make sense?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got it
Correct Vagrant SSH formatting to be consistent with the overall formatting standards Resolved #163
Add shorthand command for
ssh -p 2222 vagrant@localhost
vagrant ssh
#162 Documentation - SSH Shorthand