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

Add go.mod to the repository #502

Merged
merged 3 commits into from
Dec 5, 2019
Merged

Add go.mod to the repository #502

merged 3 commits into from
Dec 5, 2019

Conversation

hyangah
Copy link
Contributor

@hyangah hyangah commented Dec 5, 2019

Update travis config to tests both in GOPATH mode and modules mode.
While we are here, also update go versions used in travis-ci.

AppVeyor sets the latest go stack in C:\go and that is GOROOT.
Cloning this repo under C:\go\src\.. directory confused go build
(as discussed in golang/go#34657). The fix in go is not yet
released. Thus this commit changes GOPATH to c:\gopath and clones
the repo under the directory.

Also, this commit removes the go get commands intended to pull in
dependencies. In modules mode, `go build` pulls in the required
dependencies.
@codecov-io
Copy link

codecov-io commented Dec 5, 2019

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #502   +/-   ##
======================================
  Coverage    67.1%   67.1%           
======================================
  Files          37      37           
  Lines        7604    7604           
======================================
  Hits         5103    5103           
  Misses       2097    2097           
  Partials      404     404

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 27840ff...8ba3b6f. Read the comment docs.

@aalexand aalexand merged commit f9b734f into google:master Dec 5, 2019
gmarin13 pushed a commit to gmarin13/pprof that referenced this pull request Dec 17, 2020
* Add go.mod/go.sum

* travis: update to test in modules mode and with go1.13

Remove go1.11 setup

* fix appveyor config

AppVeyor sets the latest go stack in C:\go and that is GOROOT.
Cloning this repo under C:\go\src\.. directory confused go build
(as discussed in golang/go#34657). The fix in go is not yet
released. Thus this commit changes GOPATH to c:\gopath and clones
the repo under the directory.

Also, this commit removes the go get commands intended to pull in
dependencies. In modules mode, `go build` pulls in the required
dependencies.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants