-
Notifications
You must be signed in to change notification settings - Fork 6
[#1] Switch from dep to go modules #6
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me,
Makefile
Outdated
@@ -220,19 +220,19 @@ native: peer orderer configtxgen cryptogen idemixgen configtxlator discover | |||
|
|||
linter: check-deps buildenv | |||
@echo "LINT: Running code checks.." | |||
@$(DRUN) $(DOCKER_NS)/fabric-buildenv:$(DOCKER_TAG) ./scripts/golinter.sh | |||
@$(DRUN) -e GOPROXY=$(GOPROXY) $(DOCKER_NS)/fabric-buildenv:$(DOCKER_TAG) ./scripts/golinter.sh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this imply that each Makefile step is fetching Go modules?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
each makefile step will fetch go module if not exist in mod folder in $GOPATH
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is another instance in another script.
12d3df0
to
0984a53
Compare
@fqutishat note: |
- remove go dep - remove vendor - add go modules Signed-off-by: Firas Qutishat <firas.qutishat@securekey.com>
I remove change-id |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
verify |
1 similar comment
verify |
Change-Id: I06d450bbe89822729327162db91118e0b595c17a
Signed-off-by: Firas Qutishat firas.qutishat@securekey.com