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

Add ocp-index-top #123

Merged
merged 2 commits into from
Jan 4, 2018
Merged

Add ocp-index-top #123

merged 2 commits into from
Jan 4, 2018

Conversation

reynir
Copy link
Contributor

@reynir reynir commented Nov 1, 2017

Ocp-index-top implements a toplevel directive #doc that can be used to lookup documentation using ocp-index's LibIndex module.

It is written by myself. It's a bit of a hack, and there's an issue with utop since utop 2.0.1 (see reynir/ocp-index-top#9). Because of the issues it's perhaps not 'awesome' enough, but I find it quite useful myself when exploring things in the toplevel.

@gasche
Copy link
Collaborator

gasche commented Nov 1, 2017

I like the idea, but maybe we/you could restructure the list to have one ocp-index entry, which also points out to satellite projects, namely ocp-browser and ocp-index-top? (In the free text or as a sub-list.) This is already what we did for Menhir and Oasis for example.

@reynir
Copy link
Contributor Author

reynir commented Nov 1, 2017

I'm fine with that!

@Khady
Copy link
Contributor

Khady commented Nov 1, 2017 via email

@rizo
Copy link
Member

rizo commented Jan 4, 2018

@reynir ocp-index-top is very handy, thanks for submitting the PR!

I rearranged the structure as suggested by @gasche and mentioned ocp-grep (there's no separate link for it unfortunately).

@rizo rizo merged commit b34c997 into ocaml-community:master Jan 4, 2018
@reynir
Copy link
Contributor Author

reynir commented Jan 4, 2018

Hi @rizo, thank you for taking initiative!

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.

4 participants