Skip to content
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

Add sshm package #1501

Merged
merged 1 commit into from
Jul 19, 2021
Merged

Add sshm package #1501

merged 1 commit into from
Jul 19, 2021

Conversation

nitrocode
Copy link
Member

@nitrocode nitrocode commented Jul 19, 2021

what

  • Add sshm package

why

  • Easier to use ssm

references

commands

make init
make readme
make -C .github workflows
make -C vendor/sshm install
make -C vendor/sshm test

Then locally on osx

$ sshm --version
flag provided but not defined: -version
Usage of sshm:
  -i string
    	InstanceID for direct connection
  -lpn string
    	Local Port Number for Proxy
  -p string
    	Profile from ~/.aws/config
  -pn string
    	Port Number for Proxy
  -r string
    	Region, default is eu-west-1

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thank you for submitting this PR! If you haven't already joined our slack community, then we invite you to do so.

We receive an overwhelming number of contributions. By joining our slack, we'll be able to review your PR faster.

Slack Community

@osterman
Copy link
Member

Missing workflow for sshm. Run make -C .github workflows

@nitrocode nitrocode marked this pull request as ready for review July 19, 2021 21:08
@nitrocode nitrocode requested review from a team as code owners July 19, 2021 21:08
@osterman osterman merged commit 35494ea into master Jul 19, 2021
@osterman osterman deleted the sshm branch July 19, 2021 23:30
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.

2 participants