-
Notifications
You must be signed in to change notification settings - Fork 3
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
Upgrade gorilla handlers package to v1.3.0 #35
Upgrade gorilla handlers package to v1.3.0 #35
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.
Why are we removing the "vegeta" load test?
The load test was not being run as part of the CI workflow since it was added to the file two and a half years ago, so I don't see any value in keeping it. And also the service should be deprecated at some point anyway in favour of |
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.
Shouldn't we bump gorilla handlers to the latest minor version which is v1.4.2
16345f7
to
c4c056a
Compare
Also migrate the project to modules and using the golang-ci orb.