Skip to content

docs: remove old doxygen builds #24743

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

mrpollo
Copy link
Contributor

@mrpollo mrpollo commented Apr 21, 2025

hopefully i got them all and the right way

Signed-off-by: Ramon Roche <mrpollo@gmail.com>
@mrpollo mrpollo requested a review from dagar April 21, 2025 18:14
@hamishwillee
Copy link
Contributor

Just FMI, why? Is the generated documentation not useful?

@mrpollo
Copy link
Contributor Author

mrpollo commented May 6, 2025

As far as I'm aware, we are not using any of this, are we? please confirm @hamishwillee @dagar

@hamishwillee
Copy link
Contributor

As far as I'm aware, we are not using any of this, are we? please confirm @hamishwillee

I can tell you it isn't linked anywhere from PX4 docs.

Doxygen is an excellent tool if you want to publish and maintain docs for a public API/interface. So if PX4 were to more obviously manage its interfaces this would be a good thing to keep. If we're not using it for that purpose, it isn't needed - and we don't.
Probably someone will try add it back at some point because it usually considered a good thing to do.

@DronecodeBot
Copy link

This pull request has been mentioned on Discussion Forum for PX4, Pixhawk, QGroundControl, MAVSDK, MAVLink. There might be relevant details there:

https://discuss.px4.io/t/px4-team-sync-and-community-q-a-may-6-2025/45433/2

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