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

CONTRIB: Automate update of API documentation #10333

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tvegas1
Copy link
Contributor

@tvegas1 tvegas1 commented Nov 26, 2024

What?

Move steps from wiki to script to help avoid/fix mistakes. Wiki will need to be updated.

How?

./contrib/api_update.sh upstream/v1.18.x
git push upstream HEAD:refs/heads/gh-pages.v1.18.0-rc1

@tvegas1 tvegas1 force-pushed the automate_update_api_doc branch from 4507466 to 9106af0 Compare November 26, 2024 14:50
rakhmets
rakhmets previously approved these changes Nov 28, 2024
Comment on lines +26 to +27
git clean -xdf
git reset --hard
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it looks dangerous. Maybe mention it explicitly/(more clear) on line 18

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added check to address comment


set -x

sudo yum -y install doxygen doxygen-latex
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe also add support for ubuntu/debian?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@tvegas1 tvegas1 force-pushed the automate_update_api_doc branch from c808e5f to e2261e3 Compare November 29, 2024 10:23
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.

3 participants