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

Set module version to v1 #319

Merged
merged 1 commit into from
Oct 24, 2022
Merged

Set module version to v1 #319

merged 1 commit into from
Oct 24, 2022

Conversation

feedmeapples
Copy link
Contributor

@feedmeapples feedmeapples commented Oct 17, 2022

What was changed

Set the mod version to v1

Why?

As we are not currently releasing v2 the mod version conflicts with the github release version, resulting in not being able to import the module in other projects

Checklist

  1. Closes

  2. How was this tested:

unit tests

  1. Any docs updates needed?

// Completed - green
// Started - blue
// Others - default (white/black)
//
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this extra line is now added by Go plugin automatically

@mindaugasrukas
Copy link

Is this some kind of breaking change?

@feedmeapples
Copy link
Contributor Author

Is this some kind of breaking change?

as it was broken with /v2 as an import, it couldn't be used. This is a breaking change though should fix the importing for the actual use

@feedmeapples feedmeapples merged commit d40cb65 into main Oct 24, 2022
@feedmeapples feedmeapples deleted the mod-v1 branch October 24, 2022 20:08
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.

3 participants