-
Notifications
You must be signed in to change notification settings - Fork 70
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
Fallback to ocaml.org documentation page #1331
Conversation
cc @sabine does it sound like a reasonable idea? |
5f4524a
to
9bfe28d
Compare
This PR also needs a change log entry! |
@tmattio do you have some feedback on this one? |
@tatchi Could you add a change log entry? |
These URLs are stable, so linking to them is fine. Sure, we can't guarantee the docs built on OCaml.org, but since it's a fallback that should be fine! @smorimoto |
Thanks 🙌 |
done (sorry for the delay) |
e95b8bb
to
489f6a0
Compare
Fallback to the
ocaml.org
package documentation page if there is nodoc
field in the package's opam file. Note that it doesn't currently check whether the page exists onocaml.org
, so it may return a404
error.Screen.Recording.2024-01-05.at.16.52.32.mov