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

Docs: Clarify that visitors are unavailable under go. #2014

Closed
wants to merge 2 commits into from
Closed

Docs: Clarify that visitors are unavailable under go. #2014

wants to merge 2 commits into from

Conversation

savv
Copy link

@savv savv commented Sep 16, 2017


This change is Reviewable

@savv
Copy link
Author

savv commented Sep 16, 2017

@cyberfox

@cyberfox
Copy link

cyberfox commented Oct 1, 2017

@savvopoulos It makes sense to document, but if we could get a pull request through, I'd rather it be the one that actually implements it. I'm also not sure if, paradoxically, this change would conflict with #1841 and make it harder to land. :-/

@savv savv closed this Oct 6, 2017
@savv savv deleted the patch-1 branch October 6, 2017 07:05
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