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

Add missing documentation for winch/gripper plugins #311

Merged

Conversation

potaito
Copy link
Contributor

@potaito potaito commented Jan 31, 2023

Slipped through the cracks in the previous PR: #308

Addresses the following CI failure over in MAVSDK:

2023-01-31T13:50:06.1606188Z Preprocessing /home/runner/work/MAVSDK/MAVSDK/tools/docs/install/include/mavsdk/plug/home/runner/work/MAVSDK/MAVSDK/tools/docs/install/include/mavsdk/plugins/winch/winch.h:91: warning: Compound mavsdk::Winch::StatusFlags is not documented.
2023-01-31T13:50:06.1606831Z /home/runner/work/MAVSDK/MAVSDK/tools/docs/install/include/mavsdk/plugins/winch/winch.h:28: warning: Compound mavsdk::Winch is not documented.
2023-01-31T13:50:06.1607356Z /home/runner/work/MAVSDK/MAVSDK/tools/docs/install/include/mavsdk/plugins/gripper/gripper.h:28: warning: Compound mavsdk::Gripper is not documented.
2023-01-31T13:50:06.1607877Z /home/runner/work/MAVSDK/MAVSDK/tools/docs/install/include/mavsdk/plugins/winch/winch.h:128: warning: Compound mavsdk::Winch::Status is not documented.
2023-01-31T13:50:06.1608449Z Generating docs for /home/runner/work/MAVSDK/MAVSDK/tools/docs/install/include/mavsdk/plugins/gripper/gripper.h:64: warning: Member GripperAction (enumeration) of class mavsdk::Gripper is not documented.
2023-01-31T13:50:06.1609079Z Generating docs/home/runner/work/MAVSDK/MAVSDK/tools/docs/install/include/mavsdk/plugins/winch/winch.h:64: warning: Member WinchAction (enumeration) of class mavsdk::Winch is not documented.

@JonasVautherin JonasVautherin merged commit 328778a into mavlink:main Jan 31, 2023
@potaito potaito deleted the potaito/fix-winch-gripper-docs branch February 1, 2023 06:39
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.

2 participants