-
Notifications
You must be signed in to change notification settings - Fork 213
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
Publish API docs #1094
Publish API docs #1094
Conversation
2bd463a
to
acf70bb
Compare
Maybe also publish Update documentation.yml Update documentation.yml Update documentation.yml Update documentation.yml
acf70bb
to
c2936bb
Compare
Looks solid overall, though not sure exacatly how to test. I ran
Hmmmm... expected? |
That's a new error. I'll have to fix it before we merge lol |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generates good-looking content locally for me. Concur, publish will be the next step
…on into 2023-12-30_javadoc_pub
Redirects I want:
demo.photonvision.org redirected to https://photonvision.github.io/photonvision/built-client/
javadocs.photonvision.org redirected to https://photonvision.github.io/photonvision/built-docs/javadoc/
cppdocs.photonvision.org redirected to https://photonvision.github.io/photonvision/built-docs/doxygen/html/
For now this runs on all commits to master. Once we confirm it works, let's pull back to only tagged releases