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.

Sign up for GitHub

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

Jump to bottom

[20709] Remove RTPSMessageGroup from aliases-api includes #744

Merged
merged 1 commit into from
May 24, 2024
Merged

[20709] Remove RTPSMessageGroup from aliases-api includes #744

merged 1 commit into from
May 24, 2024

Conversation

elianalf
Copy link
Contributor

@elianalf elianalf commented Apr 5, 2024 •
edited by EduPonz

Description

Remove RTPSMessageGroup from aliases-api includes.
Related PR in Fast DDS:

Contributor Checklist

  • Commit messages follow the project guidelines.
  • N/A Code snippets related to the added documentation have been provided.
  • N/A Documentation tests pass locally.
  • 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 added this to the v3.0.0 milestone Apr 5, 2024
@elianalf elianalf mentioned this pull request Apr 5, 2024
Merged
7 tasks
@elianalf elianalf changed the title Refs #20709: Remove RTPSMessageGroup from aliases-api includes [20709] Remove RTPSMessageGroup from aliases-api includes Apr 5, 2024
@elianalf
Copy link
Contributor Author

elianalf commented Apr 8, 2024

@richiprosima please test_3 this

@EduPonz EduPonz force-pushed the 3.0.x-devel branch 3 times, most recently from 3fbf8f1 to 497ab7c Compare May 14, 2024 07:05
Base automatically changed from 3.0.x-devel to master May 14, 2024 07:19
@elianalf elianalf force-pushed the feature/make_private_rtps_messages branch from 58f196f to 8ee213b Compare May 21, 2024 09:07
@EduPonz
Copy link
Member

EduPonz commented May 21, 2024

@richiprosima please test_3 this

EduPonz
EduPonz previously approved these changes May 21, 2024
View reviewed changes
@elianalf
Refs #20709: Remove RTPSMessageGroup from aliases-api includes
142f2db 
Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com>
@elianalf elianalf force-pushed the feature/make_private_rtps_messages branch from 8ee213b to 142f2db Compare May 21, 2024 10:28
@elianalf
Copy link
Contributor Author

@richiprosima please test_3 this

@elianalf elianalf requested a review from EduPonz May 21, 2024 10:29
@EduPonz
Copy link
Member

EduPonz commented May 24, 2024

@richiprosima please test_3 this

@EduPonz EduPonz merged commit e9b2984 into master May 24, 2024
4 checks passed
@EduPonz EduPonz deleted the feature/make_private_rtps_messages branch May 24, 2024 06:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@EduPonz EduPonz EduPonz approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v3.0.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@elianalf @EduPonz