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

improve GitHub Actions workflows #1190

Merged
merged 2 commits into from Feb 1, 2021

Conversation

shogo82148
Copy link
Contributor

@shogo82148 shogo82148 commented Jan 31, 2021

Description

  • add windows to build matrix
  • use newer versions of MySQL and MariaDB

Checklist

  • Code compiles correctly
  • Created tests which fail without the change (if possible)
  • All tests passing
  • Extended the README / documentation, if necessary
  • Added myself / the copyright holder to the AUTHORS file

- add windows to build matrix
- use newer versions of MySQL and MariaDB
pull_request:
push:
branches:
- "master"
Copy link
Member

Choose a reason for hiding this comment

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

Why restrict the branch?

When I pushed a new branch to my fork and I want to run test, I need to edit this file.
It is cumbersome.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it comes from starter workflows https://github.com/actions/starter-workflows/blob/c59b62dee0eae1f9f368b7011cf05c2fc42cf084/ci/go.yml#L3-L7
but I agree with you, and will remove it.

@methane
Copy link
Member

methane commented Feb 1, 2021

I'm a little surprised by shell: python, and it is python2. But it's nice than bash.

"shogo82148/actions-setup-mysql@v1" looks great!
I will use it in other projects.

@shogo82148
Copy link
Contributor Author

Thank you!

I'm a little surprised by shell: python, and it is python2.

me, too.
ubuntu-20.04 will be the default soon actions/runner-images#1816, and its default python is python 3.
after that, we can use python 3 in shell: python.

@shogo82148 shogo82148 merged commit 4d5208a into go-sql-driver:master Feb 1, 2021
@shogo82148 shogo82148 deleted the improve-github-actions branch February 1, 2021 14:46
@shogo82148 shogo82148 mentioned this pull request Mar 7, 2021
andygrunwald added a commit to andygrunwald/mysql that referenced this pull request Jul 31, 2021
* master: (93 commits)
  return unsigned in database type name when necessary (go-sql-driver#1238)
  add an invalid DSN test case (go-sql-driver#1235)
  refactoring (*textRows).readRow in a more clear way (go-sql-driver#1230)
  use utf8mb4 instead of utf8 in TestCharset (go-sql-driver#1228)
  improve readability follows go-staticcheck (go-sql-driver#1227)
  support Is comparison on MySQLError (go-sql-driver#1210)
  Wording correction in README (go-sql-driver#1218)
  noCopy implements sync.Locker (go-sql-driver#1216)
  Fix readme: MaxIdle is same or less than MaxOpen (go-sql-driver#1215)
  Drop support of Go 1.12 (go-sql-driver#1211)
  Release v1.6.0 (go-sql-driver#1197)
  add Go 1.16 to the build matrix (go-sql-driver#1198)
  Implement driver.Validator interface (go-sql-driver#1174)
  handling decoding pem error (go-sql-driver#1192)
  stop rounding times (go-sql-driver#1172)
  improve GitHub Actions workflows (go-sql-driver#1190)
  Move tests from Travis to Actions (go-sql-driver#1183)
  Fix go vet error (go-sql-driver#1173)
  README: Make usage code more friendly (go-sql-driver#1170)
  Fix a broken link to cleartext client side plugin (go-sql-driver#1165)
  ...
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.

None yet

2 participants