Skip to content

Conversation

@timvaillancourt
Copy link
Collaborator

@timvaillancourt timvaillancourt commented Jun 5, 2022

Description

This PR defaults the build to go1.17.11

This resolves #1128

In case this PR introduced Go code changes:

  • contributed code is using same conventions as original code
  • script/cibuild returns with no formatting errors, build errors or unit test errors.

@timvaillancourt timvaillancourt requested review from a user and rashiq June 5, 2022 20:57
@timvaillancourt timvaillancourt added this to the v1.1.5 milestone Jun 5, 2022
@timvaillancourt timvaillancourt merged commit 308ba7f into master Jul 6, 2022
@ghost ghost modified the milestones: v1.1.5, v1.1.6 Jul 7, 2022
RainbowDashy pushed a commit to RainbowDashy/gh-ost that referenced this pull request Jul 11, 2022
* Default to go1.17.11

* `go mod vendor`

Co-authored-by: dm-2 <45519614+dm-2@users.noreply.github.com>
@timvaillancourt timvaillancourt deleted the go-1_17 branch July 13, 2022 03:17
Copy link

@Nholam1990 Nholam1990 left a comment

Choose a reason for hiding this comment

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

vendor/modules.txt

@@ -1,3 +0,0 @@
module github.com/go-sql-driver/mysql

Choose a reason for hiding this comment

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

@timvaillancourt what am I looking at here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@FromAshWeRise the files in vendor/ are the result of go mod vendor + git add vendor --all, so go mod wanted this somehow

github.com/davecgh/go-spew v1.1.0/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c=
github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
github.com/google/renameio v0.1.0/go.mod h1:KWCgfxg9yswjAJkECMjeO8J8rahYeXnNhOm40UhjYkI=

Choose a reason for hiding this comment

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

did y'all not like the name I gave it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

go binary checksums are out of date

6 participants