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

Remove unused Godeps files; update README for Go Modules #1114

Merged
merged 8 commits into from
Feb 17, 2021

Conversation

mrutkows
Copy link
Contributor

@mrutkows mrutkows commented Feb 12, 2021

The Godeps directory has not been used since updating the project to use Go Modules. This PR removes that directory structure and any README instructions referencing go dep utility (which is now deprecated). In its place, we are adding instructions on how to build and maintain the project and its dependencies using go mod and go get and discussing how contributors should prescriptively update Go module dependencies and be cognizant of using proper versioned releases for releasing this project's source code and binaries.

@mrutkows mrutkows marked this pull request as draft February 12, 2021 00:46
@mrutkows mrutkows marked this pull request as ready for review February 17, 2021 20:11
@mrutkows mrutkows requested a review from rabbah February 17, 2021 22:59
$ cd $GOPATH
$ go get github.com/apache/openwhisk-wskdeploy # see known issues below if you get an error
$ go get github.com/tools/godep # get the dependency manager
git clone [email protected]:apache/openwhisk-wskdeploy
Copy link
Member

Choose a reason for hiding this comment

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

wow, love the simplicity 👍

Copy link
Member

@pritidesai pritidesai left a comment

Choose a reason for hiding this comment

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

Thank you @mrutkows for the detailed and simple instructions on go mod 👍

@pritidesai pritidesai merged commit 7126d6c into apache:master Feb 17, 2021
@mrutkows mrutkows deleted the godeps branch February 17, 2021 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants