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

Fix modules #63

Merged
merged 1 commit into from
Jul 28, 2020
Merged

Fix modules #63

merged 1 commit into from
Jul 28, 2020

Conversation

tfussell
Copy link
Contributor

@tfussell tfussell commented Jul 28, 2020

I wasn't able to import this library with github.com/giantswarm/gsclientgen/v2 because the module definition in go.mod wasn't updated to match the major version (required for v2+). I don't see any changes in the generated API specs (generated with make, compared master with this branch using diff).

@tfussell tfussell self-assigned this Jul 28, 2020
@@ -1,19 +1,11 @@
module github.com/giantswarm/gsclientgen
module github.com/giantswarm/gsclientgen/v2
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Here is the main change.

github.com/mailru/easyjson v0.0.0-20190403194419-1ea4449da983 // indirect
golang.org/x/net v0.0.0-20190509222800-a4d6f7feada5 // indirect
golang.org/x/text v0.3.2 // indirect
github.com/go-openapi/errors v0.19.6
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Using the latest released versions for each dep.

@tfussell tfussell marked this pull request as ready for review July 28, 2020 14:18
@tfussell tfussell requested a review from a team July 28, 2020 14:18
@tfussell tfussell merged commit 82b88b1 into master Jul 28, 2020
@tfussell tfussell deleted the fix-modules branch July 28, 2020 14:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants