-
Notifications
You must be signed in to change notification settings - Fork 277
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
V2 is removed #319
Comments
Can you explain to me what broke for you? I have not tagged a release that includes #318 yet. Here are some details that would be helpful:
I suspect what might have actually broken you is that I deleted the |
Maybe the concern is whether the library can still be imported with the |
Hello,
Correct
Yes, Go modules
Go 1.21
Currently, I have no problem. But if you remove v1 or if you plan to release a new version of v1 (for example with v1.11) with the v2. It will break all my code since v2 is not compatible with v1 (v1.10 and before). |
v1 was removed from the main branch in 2019 so changes on on main should not impact v1. Any releases for v1 would happen on the The v2 subdirectory is not necessary for compatibility afaik. Popular projects like https://github.com/redis/go-redis, which is on v9, does not have a v9 directory. The v9 is just part of the name of the module as defined in the go.mod file. The only reason a v2 directory existed on this repo is because there was a period of time where both v1 and v2 co-existing on the main branch. This is no longer the case, so I think it is safe to clean up the v2 directory. If` you have evidence that this is unsafe, I am happy to reconsider. Closing this issue since it seems like nothing is actually broken right now. |
With #318 & f0cd8fa, did you expect that V2 becomes V1?
If it is true, it is not conform with semver and it is a big breaking change.
The text was updated successfully, but these errors were encountered: