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

GO111MODULE install directions #1450

Merged
merged 1 commit into from
Feb 11, 2020
Merged

GO111MODULE install directions #1450

merged 1 commit into from
Feb 11, 2020

Conversation

gdbelvin
Copy link
Contributor

The current install instructions fall back to non-module mode non-module
mode and result in incorrect dependencies.

Fixes #1390

The current install instructions fall back to non-module mode non-module
mode and result in incorrect dependencies.

Fixes google#1390
@gdbelvin gdbelvin requested a review from a team as a code owner February 11, 2020 15:28
@gdbelvin gdbelvin requested a review from RJPercival February 11, 2020 15:28
@codecov
Copy link

codecov bot commented Feb 11, 2020

Codecov Report

Merging #1450 into master will decrease coverage by 0.24%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1450      +/-   ##
==========================================
- Coverage   66.56%   66.31%   -0.25%     
==========================================
  Files          54       54              
  Lines        4026     4026              
==========================================
- Hits         2680     2670      -10     
- Misses        957      962       +5     
- Partials      389      394       +5     
Impacted Files Coverage Δ
core/sequencer/trillian_client.go 58.57% <0.00%> (-2.86%) ⬇️
core/sequencer/server.go 72.96% <0.00%> (-1.96%) ⬇️
core/integration/client_tests.go 84.77% <0.00%> (-0.47%) ⬇️

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 a9b296f...7c765a3. Read the comment docs.

@@ -33,7 +33,7 @@ development.

### Setup
1. Install [Go 1.13](https://golang.org/doc/install).
2. `go get github.com/google/keytransparency/cmd/keytransparency-client `
2. `GO111MODULE=on go get github.com/google/keytransparency/cmd/keytransparency-client`
Copy link
Contributor

Choose a reason for hiding this comment

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

If the user has installed Go 1.13, as specified in step 1, then this shouldn't be necessary should it? Nonetheless, it won't hurt.

@gdbelvin gdbelvin merged commit 128f290 into google:master Feb 11, 2020
@gdbelvin gdbelvin deleted the install branch February 18, 2020 07:56
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

missing google/tink/go/insecure
3 participants