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

AP_DDS: move closing #endif for status publisher #28597

Merged

Conversation

srmainwaring
Copy link
Contributor

Must be before the closing brace and status_ok check.

- Must be before the status_ok check.

Signed-off-by: Rhys Mainwaring <[email protected]>
@Ryanf55
Copy link
Collaborator

Ryanf55 commented Nov 12, 2024

Introduced by #28337

@peterbarker peterbarker merged commit e140458 into ArduPilot:master Nov 12, 2024
99 checks passed
@peterbarker
Copy link
Contributor

Merged, thanks!

Note that once these options are added to test_build_options.py they will undergo a minimal amount of testing as part of CI.

@Ryanf55
Copy link
Collaborator

Ryanf55 commented Nov 12, 2024

Merged, thanks!

Note that once these options are added to test_build_options.py they will undergo a minimal amount of testing as part of CI.

Yep, we're super close. Just have a linker error in this branch, otherwise it's good to go.
#28440

@srmainwaring srmainwaring deleted the prs/pr-dds-fix-status-ifdef branch November 13, 2024 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants