Skip to content

Conversation

@JoC2000
Copy link
Contributor

@JoC2000 JoC2000 commented Feb 10, 2026

  • Updated links that referenced old source files location.
  • Updated PX4 documentation links from master to main.
  • Updated QGC links.
  • Modified .gitignore to new docs location.

- Updated links that referenced old source files location.
- Updated PX4 documentation links from master to main.
- Updated QGC links.
- Modified .gitignore to new docs location.
JonasVautherin
JonasVautherin previously approved these changes Feb 10, 2026
Copy link
Collaborator

@JonasVautherin JonasVautherin left a comment

Choose a reason for hiding this comment

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

Oh nice, thanks a lot!

@julianoes: macOS seems unrelated (flaky test apparently) and the docs check... I don't know. Seems to complain because of a change? Not sure how it works for md files. It doesn't look like a style issue, it seems like it complains just because there is a diff.

@julianoes
Copy link
Collaborator

I'm going to try to just run the docs generation script on top of it.

@JoC2000
Copy link
Contributor Author

JoC2000 commented Feb 11, 2026

@julianoes what is the issue with those changes? For action and camera servers the changes were made because the current links don't work.

@julianoes
Copy link
Collaborator

@JoC2000 if you want to change these, it has to come from https://github.com/mavlink/MAVSDK-Proto and then generated.

See https://mavsdk.mavlink.io/main/en/cpp/contributing/autogen.html

@JoC2000
Copy link
Contributor Author

JoC2000 commented Feb 11, 2026

Understood! I'll take a look at it. I'll get started on a separate PR over Proto then.

@JoC2000
Copy link
Contributor Author

JoC2000 commented Feb 11, 2026

I found more broken links after checking the autogen docs and fixed the contributing section links.

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