Skip to content
This repository was archived by the owner on Oct 11, 2024. It is now read-only.

Fix scripts to not depend on KT being in $GOPATH #1428

Merged
merged 2 commits into from
Jan 8, 2020

Conversation

RJPercival
Copy link
Contributor

Since Go Modules became supported by KT, it should also support being checked out to an arbitrary location.

Since Go Modules became supported by KT, it should also support KT
being checked out to an arbitrary location.
@RJPercival RJPercival added the bug label Jan 8, 2020
@RJPercival RJPercival requested a review from gdbelvin January 8, 2020 12:53
@RJPercival RJPercival requested a review from a team as a code owner January 8, 2020 12:53
@codecov
Copy link

codecov bot commented Jan 8, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1428   +/-   ##
=======================================
  Coverage   66.31%   66.31%           
=======================================
  Files          54       54           
  Lines        4026     4026           
=======================================
  Hits         2670     2670           
  Misses        962      962           
  Partials      394      394

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 1e6b68d...12be28c. Read the comment docs.

@gdbelvin gdbelvin merged commit ceecf10 into google:master Jan 8, 2020
gdbelvin added a commit to gdbelvin/keytransparency that referenced this pull request Jan 8, 2020
* master:
  Fix scripts to not depend on KT being in $GOPATH (google#1428)
  Fix permissions on Prometheus config files (google#1429)
@RJPercival RJPercival deleted the scripts-modules branch January 8, 2020 14:28
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.

3 participants