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

Module path not updated to github.com/gorilla/websocket/v2 and new tag does not follow semantic versioning #860

Closed
1 task done
FZambia opened this issue Nov 4, 2023 · 2 comments
Labels

Comments

@FZambia
Copy link
Contributor

FZambia commented Nov 4, 2023

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

In yesterday's v2 release I noticed two issues:

  1. Path should be updated in go.mod. See https://go.dev/blog/v2-go-modules and specifically:
If an old package and a new package have the same import path,
the new package must be backwards compatible with the old package.
  1. New v2.0 tag does not follow semantic versioning - please take a look how all the previous tags were named.

Expected Behavior

When releasing v2 version of module it's recommended to also update module path. So I suppose it must be:

github.com/gorilla/websocket/v2

And tag is v2.0.0

Steps To Reproduce

module github.com/gorilla/websocket

https://github.com/gorilla/websocket/tree/v2.0

Anything else?

I suppose this may cause issues with Go tooling.

Also, could you please tell why there is vendor directory appeared in repo root? It's quite strange to see it in 2023.

@FZambia FZambia added the bug label Nov 4, 2023
@FZambia
Copy link
Contributor Author

FZambia commented Nov 4, 2023

Never used it myself, but probably may be useful - retract directive of go.mod

@coreydaley
Copy link
Contributor

This should be fixed now, release of gorilla/websocket is v1.5.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: ✅ Done
Development

No branches or pull requests

2 participants