-
Notifications
You must be signed in to change notification settings - Fork 98
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
Include PPX by example to the manual #522
Comments
@NathanReb, I'm getting back to work on
As you said about figuring out those details, do you have something in mind about how to integrate this? |
Glad to hear that @pedrobslisboa ! Ideally it'd be great to have the content of your We already have an example directory so I was hoping the actual code and dune files could simply be moved there, each in its own subdir, following the structure you have already in ppx-by-example. The links will have to updated, for instance the links to the ppxlib documentation can be turned into internal links. If I remember correctly there are also links to the source code directly on github, which is great. The problem for those is that they can grow old pretty quickly once we publish the manual, or on older versions of our documentation on ocaml.org. I don't think it's too much of a problem as long as the latest version is up-to-date. Having this great ressource is already a huge benefit! We can go step by step here, for instance we could leave the |
One thing we could consider too is using MDX to check code fragments that live inside the documentation to make sure those don't go outdated! Again, that's not high priority and can be left for much later. |
@pedrobslisboa has been working on a great piece of documentation for ppx authors: https://github.com/pedrobslisboa/ppx-by-example.
Following a short discussion on the discuss announcement post they agreed to integrate it directly into ppxlib's manual as it would make sense to centralize all PPX related documentation and guides here.
I'm opening this issue so we can track progress on this and figure out the details on how to best integrate this.
The text was updated successfully, but these errors were encountered: