-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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: remove ioutil package #634
Conversation
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! Thank you. Apart from the signal thing, I'll sign off this commit locally and push to main branch.
@@ -201,7 +201,7 @@ func attack(opts *attackOpts) (err error) { | |||
res := atk.Attack(tr, opts.rate, opts.duration, opts.name) | |||
enc := vegeta.NewEncoder(out) | |||
sig := make(chan os.Signal, 1) | |||
signal.Notify(sig, os.Interrupt) | |||
signal.Notify(sig, os.Interrupt, syscall.SIGTERM) |
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.
This change seems unrelated. Accidental?
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.
I think SIGTERM is also commonly used, so I add it. Can we add it in this pr? If not, I will remove it.
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.
It's fine.
ioutil is deprecated, see https://go.dev/doc/go1.16#ioutil Follow up on tsenart#634 Signed-off-by: Alexander Yastrebov <[email protected]>
ioutil is deprecated, see https://go.dev/doc/go1.16#ioutil Follow up on tsenart#634 Signed-off-by: Alexander Yastrebov <[email protected]>
ioutil is deprecated, see https://go.dev/doc/go1.16#ioutil Follow up on #634 Signed-off-by: Alexander Yastrebov <[email protected]> Signed-off-by: Tomás Senart <[email protected]>
Background
ioutil package is not recommand to use, instead use io package and os package
Checklist