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.

Sign up for GitHub

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

Jump to bottom

Protocol.md doesn't describe what th field 'notifications' is for (and other minor issues like that) #680

Closed
Kakadu opened this issue Jul 24, 2017 · 3 comments
Closed

Protocol.md doesn't describe what th field 'notifications' is for (and other minor issues like that) #680

Kakadu opened this issue Jul 24, 2017 · 3 comments

Comments

@Kakadu
Copy link

Kakadu commented Jul 24, 2017 •
edited

https://github.com/ocaml/merlin/blob/master/doc/dev/PROTOCOL.md#answers
@let-def
Copy link
Contributor

let-def commented Jul 24, 2017

Thanks, I just updated the documentation.

@Kakadu
Copy link
Author

Kakadu commented Jul 24, 2017 •
edited

@let-def Also "errors" command returns a field like

"sub": [],

which is not described too.

https://github.com/ocaml/merlin/blob/master/doc/dev/PROTOCOL.md#errors

@Kakadu Kakadu changed the title Protocol.md doesn't describe what th field 'notifications' is for Protocol.md doesn't describe what th field 'notifications' is for (and other minor issues like that) Jul 24, 2017
@let-def
Copy link
Contributor

let-def commented Aug 9, 2017

Thanks, I added a few words about sub (you shouldn't care about it for now, it is experimental).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Kakadu @let-def