Skip to content
This repository has been archived by the owner on Apr 12, 2019. It is now read-only.

Migrate to go mod #134

Closed
wants to merge 5 commits into from
Closed

Migrate to go mod #134

wants to merge 5 commits into from

Conversation

sapk
Copy link
Member

@sapk sapk commented Nov 12, 2018

This PR migrate from dep to go mod command to manage deps.
I would also propose to remove the vendor folder since this package is a lib and the deps are already vendored in gitea.

@codecov-io
Copy link

codecov-io commented Nov 12, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #134   +/-   ##
=======================================
  Coverage   33.76%   33.76%           
=======================================
  Files          28       28           
  Lines        1833     1833           
=======================================
  Hits          619      619           
  Misses       1134     1134           
  Partials       80       80

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 fbe468c...0108bcf. Read the comment docs.

@lafriks
Copy link
Member

lafriks commented Nov 12, 2018

I'm fine with removing vendor folder for this lib

@sapk
Copy link
Member Author

sapk commented Nov 12, 2018

@lafriks done 😄

@lunny
Copy link
Member

lunny commented Mar 26, 2019

go-gitea/gitea#6364 will replace this.

@sapk sapk closed this Mar 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants