Skip to content

Conversation

@regeda
Copy link
Contributor

@regeda regeda commented Dec 11, 2019

I created the PR in order to make the library godoc very helpful:

  1. All examples appear in the godoc.
  2. Examples use DialURL instead of deprecated Dial.
  3. LDAP RFC spec was removed from go files. The spec is open and observable from Internet.

Copy link
Member

@johnweldon johnweldon left a comment

Choose a reason for hiding this comment

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

Thanks for this PR.

This will resolve #253.

I don't want to lose all that code documentation entirely - what do you think about consolidating it all in a documentation text file?

@regeda
Copy link
Contributor Author

regeda commented Dec 13, 2019

@johnweldon thanks for the review.

I've added links to specifications the library implements. I don't see any reason to copy the content of specifications into go files.

@regeda regeda force-pushed the update-documentation branch from b252037 to 1bf3a08 Compare December 13, 2019 09:59
@regeda regeda force-pushed the update-documentation branch from 1bf3a08 to 62a535c Compare December 13, 2019 10:07
Copy link
Member

@johnweldon johnweldon left a comment

Choose a reason for hiding this comment

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

Love this - Thank you!

@johnweldon johnweldon merged commit a75d3c9 into go-ldap:master Dec 13, 2019
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