-
Notifications
You must be signed in to change notification settings - Fork 445
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
Renovations #294
Renovations #294
Conversation
- move to github.com/urface/cli/v2 API - refresh all dependencies to @latest - golang.org/x/crypto/ssh/terminal moved to golang.org/x/term
- deprecated functions - copylocks on range - t.Fatal in non-test goroutines - use bytes.Equal - #nosec for math/crypto
@sirupsen 👋 hiya — I'm not sure who still maintains toxiproxy, but as a recent committer is this something you might be able to review? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks, is someone able to merge so I can pull this in from master? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work. Thank you very much.
In #294 introduced a breaking change in client argument parsing. It requires to write [flags before arguments](https://github.com/urfave/cli/blob/master/docs/migrate-v1-to-v2.md#flags-before-args). Update help texts and documentation to have POSIX way to use flags and arguments.
In #294 introduced a breaking change in client argument parsing. It requires to write [flags before arguments](https://github.com/urfave/cli/blob/master/docs/migrate-v1-to-v2.md#flags-before-args). Update help texts and documentation to have POSIX way to use flags and arguments.
In #294 introduced a breaking change in client argument parsing. It requires to write [flags before arguments](https://github.com/urfave/cli/blob/master/docs/migrate-v1-to-v2.md#flags-before-args). Update help texts and documentation to have POSIX way to use flags and arguments. Co-authored-by: Drew Matheson <DAmatheson@users.noreply.github.com>
renovate dependencies
gofmt -s -w
fix some minor govet/staticheck failures