The Wayback Machine - https://web.archive.org/web/20201020110536/https://github.com/go-sql-driver/mysql/pull/1047
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

Release v1.5.0 #1047

Merged
merged 2 commits into from Jan 7, 2020
Merged

Release v1.5.0 #1047

merged 2 commits into from Jan 7, 2020

Conversation

@julienschmidt
Copy link
Member

@julienschmidt julienschmidt commented Jan 3, 2020

Blockers:

See also Milestone v1.5.0

@julienschmidt julienschmidt added this to the v1.5.0 milestone Jan 3, 2020
@julienschmidt julienschmidt mentioned this pull request Jan 3, 2020
@julienschmidt
Copy link
Member Author

@julienschmidt julienschmidt commented Jan 4, 2020

Should the build tags of conncheck rather be opt-in instead of op-out?

Current build tag: // +build !windows,!appengine

This might fail to compile on some platforms, or not?

@julienschmidt julienschmidt force-pushed the changelog branch from a3c1b1b to a0933a6 Jan 4, 2020
@shogo82148
Copy link
Contributor

@shogo82148 shogo82148 commented Jan 5, 2020

you are right. it fails on plan9.

I wrote very simple compile checker. https://gist.github.com/shogo82148/a7a61b7a1652bff07fc0dfd07ceb2122

The following is an abstract of the result for current master branch b66d043

$ENV{GOOS}="plan9"; $ENV{GOARCH}="amd64";
# github.com/go-sql-driver/mysql
./conncheck.go:42:15: undefined: syscall.EAGAIN
./conncheck.go:42:40: undefined: syscall.EWOULDBLOCK
@shogo82148 shogo82148 mentioned this pull request Jan 5, 2020
5 of 5 tasks complete
@julienschmidt
Copy link
Member Author

@julienschmidt julienschmidt commented Jan 5, 2020

I wrote very simple compile checker. https://gist.github.com/shogo82148/a7a61b7a1652bff07fc0dfd07ceb2122

We should maybe add a Travis job for that

@julienschmidt julienschmidt force-pushed the changelog branch 4 times, most recently from 2c35580 to a3c4d20 Jan 6, 2020
@vmg
Copy link
Contributor

@vmg vmg commented Jan 7, 2020

Hi @julienschmidt! Just came back from my holidays. How can I help with the release process? I see you already wrote the changelog notes. 🙇

@julienschmidt julienschmidt force-pushed the changelog branch from a3c4d20 to 89793b4 Jan 7, 2020
@julienschmidt
Copy link
Member Author

@julienschmidt julienschmidt commented Jan 7, 2020

#1054 and this PR still need a review and must be merged. Other than that, I think we're ready to tag the release. 🎉

@vmg
vmg approved these changes Jan 7, 2020
Copy link
Contributor

@vmg vmg left a comment

🎉

@julienschmidt julienschmidt merged commit 17ef3dd into master Jan 7, 2020
4 checks passed
4 checks passed
WIP Ready for review
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.1%) to 81.469%
Details
@julienschmidt julienschmidt deleted the changelog branch Jan 7, 2020
tz70s added a commit to tz70s/mysql that referenced this pull request Sep 5, 2020
* CHANGELOG: include v1.4.1

* Release v1.5.0
tz70s added a commit to tz70s/mysql that referenced this pull request Sep 5, 2020
* CHANGELOG: include v1.4.1

* Release v1.5.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.