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 info command #54

Merged
merged 5 commits into from
Mar 31, 2021
Merged

Add info command #54

merged 5 commits into from
Mar 31, 2021

Conversation

patricksanders
Copy link
Contributor

This PR adds a weep info command that prints compressed and base64-encoded debug info to help with support cases. The command is hidden from the help output since it doesn't need to be discoverable outside of support interactions.

Folks supporting Weep can also use weep info -d to decode the output from a user's info command. It will accept input via pipe:

# Pipe straight from Weep
weep info | weep info -d

# Pipe from env var
OUTPUT=$(weep info)
echo $OUTPUT | weep info -d

# Copypasta
weep info | pbcopy
pbpaste | weep info -d

Or via an argument:

OUTPUT=$(weep info)
weep info -d $OUTPUT

@patricksanders patricksanders marked this pull request as ready for review March 29, 2021 23:44
cmd/list.go Outdated
}
roles, err := client.Roles(showAll)
roles, err := client.Roles(all)
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we should just show all roles by default? The difference between "console" and "app" roles could be confusing to users

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, that does seem to get folks. I'll make that change.

@patricksanders patricksanders merged commit d5f836b into master Mar 31, 2021
@patricksanders patricksanders deleted the feat/info-command branch March 31, 2021 22:19
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