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

Update build #2551

Merged
merged 1 commit into from
Apr 22, 2021
Merged

Update build #2551

merged 1 commit into from
Apr 22, 2021

Conversation

SuperQ
Copy link
Member

@SuperQ SuperQ commented Apr 22, 2021

  • Drop /vendor.
  • Update tests to Go 1.15.
  • Update djfarrelly/maildev to 1.1.0.
  • Update mixin test for Go 1.16.
  • Bump Go modules.

Signed-off-by: SuperQ [email protected]

@SuperQ SuperQ requested a review from w0rm April 22, 2021 08:21
@SuperQ SuperQ force-pushed the superq/bump_stuff branch 4 times, most recently from 05c22ec to 7eab455 Compare April 22, 2021 09:25
@SuperQ
Copy link
Member Author

SuperQ commented Apr 22, 2021

For some reason, the acceptance tests fail with Go 1.16. Only bumping to 1.15 for now.

@SuperQ
Copy link
Member Author

SuperQ commented Apr 22, 2021

Also, there have been some changes to prometheus/common, so I've held that back.

@roidelapluie
Copy link
Member

You can rebase.

@SuperQ SuperQ force-pushed the superq/bump_stuff branch 5 times, most recently from f7a8b44 to d9c6e8b Compare April 22, 2021 11:00
* Drop /vendor.
* Update Go to 1.16.
* Update djfarrelly/maildev to 1.1.0.
* Update protoc to 3.15.8.
* Update mixin test for Go 1.16.
* Bump Go modules.

Signed-off-by: SuperQ <[email protected]>
@SuperQ SuperQ force-pushed the superq/bump_stuff branch from d9c6e8b to 99f64e9 Compare April 22, 2021 11:11
@SuperQ SuperQ requested a review from roidelapluie April 22, 2021 12:16
@SuperQ
Copy link
Member Author

SuperQ commented Apr 22, 2021

Ready for review finally.

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