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 now whoami subcommand #587

Merged
merged 1 commit into from May 26, 2017
Merged

add now whoami subcommand #587

merged 1 commit into from May 26, 2017

Conversation

TooTallNate
Copy link
Member

Not logged in:

$ now whoami
> Not currently logged in! Please run `now --login`.

Logged in, stdout is a TTY:

$ now whoami
> tootallnate
$ now switch zeit
$ now whoami
> zeit

Stdout is NOT a TTY:

$ echo "Logged in to Now as $(now whoami)"
Logged in to Now as tootallnate

Closes #585.

Not logged in:

```
$ now whoami
> Not currently logged in! Please run `now --login`.
```

Logged in, stdout is a TTY:

```
$ now whoami
> tootallnate
$ now switch zeit
$ now whoami
> zeit
```

Stdout is NOT a TTY:

```
$ echo "Logged in to Now as $(now whoami)"
Logged in to Now as tootallnate
```

Closes #585.
@matheuss
Copy link
Member

Amazing @TooTallNate 😍

I was discussing that with @rauchg the other day and I think we want now info as the main command and now whoami as an alias 👍

So can you change it to info and then document whoami as the alias? Then later I can add more info to info, like current plan etc 🙌

@rauchg
Copy link
Member

rauchg commented May 26, 2017

No, I think whoami like this is perfect

@rauchg rauchg merged commit 44f5b9f into master May 26, 2017
@rauchg rauchg deleted the add/now-whoami-subcommand branch May 26, 2017 21:22
rauchg added a commit that referenced this pull request May 26, 2017
rauchg added a commit that referenced this pull request May 26, 2017
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.

None yet

3 participants