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

[20817] Add CDRMessage_t header #138

Merged
merged 1 commit into from
May 30, 2024
Merged

Conversation

elianalf
Copy link
Contributor

@elianalf elianalf commented May 24, 2024

Description

This PR adds CDRMessage_t.h to ShapesDemo to fix a compilation error introduced from the
related implementation PR:

Contributor Checklist

  • Commit messages follow the project guidelines.
  • Changes do not break current interoperability.
  • N/A New feature has been documented/Current behavior is correctly described in the documentation.
  • N/A Applicable backports have been included in the description.

Reviewer Checklist

  • The PR has a milestone assigned.
  • The title and description correctly express the PR's purpose.
  • Check contributor checklist is correct.
  • CI passes without warnings or errors.

@elianalf elianalf force-pushed the feature/remove_miscellaneous_api branch from 7a9255c to 33360f0 Compare May 29, 2024 09:23
@elianalf elianalf added the ready-to-merge Ready to be merged. CI and changes have been reviewed and approved. label May 29, 2024
@EduPonz EduPonz merged commit 34504b6 into master May 30, 2024
4 checks passed
@EduPonz EduPonz deleted the feature/remove_miscellaneous_api branch May 30, 2024 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Ready to be merged. CI and changes have been reviewed and approved.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants