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

vendor: Switch from dep to Go 1.11 modules #6232

Closed
wants to merge 1 commit into from

Conversation

vadorovsky
Copy link
Member

@vadorovsky vadorovsky commented Nov 13, 2018

Go 1.11 introduces the feature of modules which allow to develop
Go programs without GOPATH and fetch versioned Go dependencies.

The long term intention of Go modules is to stop using vendor/
dir, although it can optionally still exist. This change leaves
the vendor/ dir for developers who still want to use GOPATH
and for offline builds.

Signed-off-by: Michal Rostecki mrostecki@suse.de


This change is Reviewable

Go 1.11 introduces the feature of modules which allow to develop
Go programs without GOPATH and fetch versioned Go dependencies.

The long term intention of Go modules is to stop using `vendor/`
dir, although it can optionally still exist. This change leaves
the `vendor/` dir for developers who still want to use GOPATH
and for offline builds.

Signed-off-by: Michal Rostecki <mrostecki@suse.de>
@vadorovsky vadorovsky requested review from a team as code owners November 13, 2018 14:58
@vadorovsky
Copy link
Member Author

test-me-please

Copy link
Member

@aanm aanm left a comment

Choose a reason for hiding this comment

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

@mrostecki I hope you didn't spend too much time with this. Unfortunately go modules are not well suited as they are right now as they rely that all golang projects to make use of semver so we can't easily correlate with kubernetes version we are using.

More info:
#5367 (comment)

golang/go#27271

@vadorovsky
Copy link
Member Author

@aanm I didn't spend much time on it. Good point, I completely missed the fact that k8s versions are not processed correctly.

I will put it on hold till Go 1.12. golang/go#25898

@vadorovsky vadorovsky closed this Nov 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants