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

Fix panic on step oauth --help #716

Merged
merged 3 commits into from Aug 9, 2022
Merged

Fix panic on step oauth --help #716

merged 3 commits into from Aug 9, 2022

Conversation

maraino
Copy link
Collaborator

@maraino maraino commented Aug 8, 2022

Description

This PR fixes a panic printing the help of the console-flow flag.

Fixes #706

@maraino maraino added this to the Next Release milestone Aug 8, 2022
@maraino maraino requested a review from dopey August 8, 2022 23:14
@github-actions github-actions bot added the needs triage Waiting for discussion / prioritization by team label Aug 8, 2022
Copy link
Contributor

@dopey dopey left a comment

Choose a reason for hiding this comment

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

lgtm. But I thought @hslatman already had a PR to fix this as well.

@dopey
Copy link
Contributor

dopey commented Aug 9, 2022

#707

@hslatman
Copy link
Member

hslatman commented Aug 9, 2022

Totally forgot about mine. It seems @maraino did some more formatting changes in this one, so I'm good if this gets merged.

@maraino maraino merged commit ed77956 into master Aug 9, 2022
@maraino maraino deleted the fix-oauth-help branch August 9, 2022 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs triage Waiting for discussion / prioritization by team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: step oauth --help crashes
3 participants