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

Update go.mod #241

Merged
merged 2 commits into from
Nov 7, 2019
Merged

Update go.mod #241

merged 2 commits into from
Nov 7, 2019

Conversation

jefferai
Copy link
Contributor

@jefferai jefferai commented Nov 5, 2019

I believe that this (plus a new tag) is what is necessary for native go modules support -- right now go modules complains that the version tag is >= v2 but the module doesn't claim to be a version >= v2.

Fixes #240, I think.

I believe that this (plus a new tag) is what is necessary for native go modules support -- right now go modules complains that the version tag is >= v2 but the module doesn't claim to be a version >= v2.
@johnweldon
Copy link
Member

Resolved by #243

@johnweldon johnweldon closed this Nov 7, 2019
@johnweldon johnweldon reopened this Nov 7, 2019
@johnweldon johnweldon mentioned this pull request Nov 7, 2019
@johnweldon johnweldon merged commit 3c7ebc5 into go-ldap:master Nov 7, 2019
@jefferai jefferai deleted the patch-1 branch November 8, 2019 15:56
@jefferai
Copy link
Contributor Author

jefferai commented Nov 8, 2019

Thank you!

hahmadia added a commit to mattermost/ldap that referenced this pull request Nov 25, 2019
* unified request flow && external binding to LDAP (go-ldap#232)

* unified request flow && external binding to LDAP

* fix debug mode

* go.mod was added

* Prevent negative waitgroup panic by `Add`ing first. (go-ldap#237)

* Use github for module name. (go-ldap#239)

* Rename asn1 ber dependency. (go-ldap#243)

* Rename asn1.ber dependency.

* go mod tidy

* Update travic CI to new asn1 ber

* Update travis go_import_path

* Update README (go-ldap#245)

* Update go.mod (go-ldap#241)

I believe that this (plus a new tag) is what is necessary for native go modules support -- right now go modules complains that the version tag is >= v2 but the module doesn't claim to be a version >= v2.

* Versioned v3 according to Go wiki, maintaining backward compatibility (go-ldap#247)

* Moved v3 to subfolder to allow for versioning >2 with go modules. Reverted top level go.mod to fix backward compatibility

* Updated readme to include directions on Go Modules, including the rationale
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.

Cut tag for current master
2 participants