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

Update IDL-Parser #2

Merged
merged 14 commits into from
Feb 18, 2024
Merged

Update IDL-Parser #2

merged 14 commits into from
Feb 18, 2024

Conversation

srmainwaring
Copy link

@srmainwaring srmainwaring commented Feb 17, 2024

Merge eProsima#74 into the ArduPilot fork in advance of upstream.

Required for:

Dependencies

@Ryanf55
Copy link
Collaborator

Ryanf55 commented Feb 17, 2024

I've updated the submodule hash, URL and specified the branch in 4ae7d0a.

@srmainwaring
Copy link
Author

I've updated the submodule hash, URL and specified the branch in 4ae7d0a.

LGTM @Ryanf55 - I've rebuilt microxrceddsgen using this branch and verified that ArduPilot/ardupilot#26095 is building, so should be good to merge.

Is there anything in CI that requires updating before this can be merged?

@Ryanf55
Copy link
Collaborator

Ryanf55 commented Feb 18, 2024

I've updated the submodule hash, URL and specified the branch in 4ae7d0a.

LGTM @Ryanf55 - I've rebuilt microxrceddsgen using this branch and verified that ArduPilot/ardupilot#26095 is building, so should be good to merge.

Is there anything in CI that requires updating before this can be merged?

I'll manually trigger a rebuild in ardupilot_docker for a new image. No code changes should be needed.

@Ryanf55 Ryanf55 merged commit 2e552e9 into ArduPilot:master Feb 18, 2024
1 check passed
@srmainwaring srmainwaring deleted the feature/v3 branch February 18, 2024 15:57
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