Skip to content
This repository has been archived by the owner on Sep 7, 2021. It is now read-only.
This repository is currently being migrated. It's locked while the migration is in progress.

update github.com/go-sql-driver/mysql to version 1.4.0 #1000

Closed
wants to merge 1 commit into from
Closed

update github.com/go-sql-driver/mysql to version 1.4.0 #1000

wants to merge 1 commit into from

Conversation

techknowlogick
Copy link
Contributor

This will eventually resolve: go-gitea/gitea#3876

@codecov-io
Copy link

codecov-io commented Jun 6, 2018

Codecov Report

Merging #1000 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1000   +/-   ##
=======================================
  Coverage   54.29%   54.29%           
=======================================
  Files          41       41           
  Lines        7548     7548           
=======================================
  Hits         4098     4098           
  Misses       2934     2934           
  Partials      516      516

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a8bd843...7df3920. Read the comment docs.

@lunny
Copy link
Member

lunny commented Jun 6, 2018

Maybe remove the vendor directory is better.

@techknowlogick
Copy link
Contributor Author

@lunny I think no because what if dependency updates in way that breaks this lib. Better to have known versions of dependency so downstream won't break upstream. Please correct me if I have a wrong though.

@techknowlogick
Copy link
Contributor Author

Closed because #1016 is better approach

@techknowlogick techknowlogick deleted the update-sql-vendor branch June 23, 2018 04:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[E] Failed to initialize ORM engine: this user requires mysql native password authentication.
3 participants