Skip to content
This repository has been archived by the owner on Jul 2, 2019. It is now read-only.

Easy2 #8

Merged
merged 13 commits into from Apr 8, 2015
Merged

Easy2 #8

merged 13 commits into from Apr 8, 2015

Conversation

Raynes
Copy link
Contributor

@Raynes Raynes commented Apr 3, 2015

No description provided.

@Raynes
Copy link
Contributor Author

Raynes commented Apr 5, 2015

I did the thing that you wished for sire. Can I gets a merge?

@@ -7,29 +7,16 @@
* For a good experience, include the region

## Installation and Updating
`npm install -g sugar-ssh`
`npm install -g easy2`
Copy link
Contributor

Choose a reason for hiding this comment

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

This is still not the right name

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yup, missed the docs, will fix

@danopia
Copy link
Contributor

danopia commented Apr 8, 2015

So is the usage sugar i-36ao8fla ssh like we talked about, or sugar ssh i-36ao8fla? I can't tell by code

@Raynes
Copy link
Contributor Author

Raynes commented Apr 8, 2015

I thought we discussed the former and decided against it and then compromised on simply allowing for the prior behavior of defaulting to sshing if someone only provides a filter?

Specifically sugar foo now behaves exactly the same as sugar ssh foo, for backwards compatibility with Sugar's simple and most common use case.

@danopia
Copy link
Contributor

danopia commented Apr 8, 2015

Yea that's fine, just fix docs please

@Raynes
Copy link
Contributor Author

Raynes commented Apr 8, 2015

Done

* `~/.aws/config` file with credentials ([details](https://github.com/aws/aws-cli#getting-started))
* For a good experience, include the region

## Installation and Updating
`npm install -g sugar-ssh`
`npm install -g sugar`
Copy link
Contributor

Choose a reason for hiding this comment

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

-ssh

@Raynes
Copy link
Contributor Author

Raynes commented Apr 8, 2015

Done.

@Raynes Raynes closed this Apr 8, 2015
@Raynes Raynes reopened this Apr 8, 2015
@Raynes
Copy link
Contributor Author

Raynes commented Apr 8, 2015

Omg buttons everywhere goodness

danopia added a commit that referenced this pull request Apr 8, 2015
@danopia danopia merged commit e3bd4ec into master Apr 8, 2015
@danopia danopia deleted the easy2 branch April 8, 2015 23:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants