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

Introduction version v2 as go module #87

Closed
wants to merge 4 commits into from
Closed

Conversation

igm
Copy link
Owner

@igm igm commented Mar 29, 2020

This PR creates a new v2 subdirectory with the content of branch v2 with addition of go.mod. Once merged into master the library can be used by importing v2 as github.com/igm/sockjs-go/v2/sockjs.

As the content matches exactly of branch v2 existing import path:

gopkg.in/igm/sockjs-go.v2/sockjs

can be safely replaced by:

github.com/igm/sockjs-go/v2/sockjs

the PR is backwards compatible and contributes to: #61

@coveralls
Copy link

coveralls commented Mar 29, 2020

Coverage Status

Coverage increased (+0.4%) to 94.92% when pulling bca8884 on go-modules-support into cd6986d on master.

@igm
Copy link
Owner Author

igm commented Apr 1, 2020

this is not valid PR. v2 will continue in branch v2 updated with go.mod.

@igm igm closed this Apr 1, 2020
@igm igm deleted the go-modules-support branch April 1, 2020 17:58
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.

2 participants