-
Notifications
You must be signed in to change notification settings - Fork 8
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
EE Getting started: add SSH forwarding #39
Conversation
ready for review |
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.
Couple of nits. Thanks @Andersson007
Co-authored-by: Don Naro <[email protected]>
Co-authored-by: Don Naro <[email protected]>
@oraNod committed, thanks, PTAL |
if no other suggestions, feel free to merge, another person confirmed that it works |
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.
@Andersson007 Sorry to badger you with more nits but just a couple things to refine. LGTM overall though. Thanks!
Co-authored-by: Don Naro <[email protected]>
Co-authored-by: Don Naro <[email protected]>
Co-authored-by: Don Naro <[email protected]>
@oraNod we make as many improvements and iterations as needed, all committed, looks much better now, thanks! |
thanks @Andersson007 |
EE Getting started: add SSH forwarding