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

[21159] Adjust for removal of include/fastrtps #810

Merged
merged 1 commit into from
Jun 19, 2024

Conversation

EduPonz
Copy link

@EduPonz EduPonz commented Jun 11, 2024

Description

This PR adjust for the include/fastrtps removal from:

It depends from the documentation changes from (docs PR not opened yet):

Contributor Checklist

  • Commit messages follow the project guidelines.
  • N/A: Code snippets related to the added documentation have been provided.
  • 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.

@EduPonz EduPonz added this to the v3.0.0 milestone Jun 11, 2024
elianalf
elianalf previously approved these changes Jun 13, 2024
Signed-off-by: eduponz <eduardoponz@eprosima.com>
@EduPonz EduPonz force-pushed the feature/cleanup_include_fastrtps branch from 0f9dfee to c7173f0 Compare June 18, 2024 09:35
@EduPonz EduPonz requested a review from elianalf June 18, 2024 09:39
@EduPonz
Copy link
Author

EduPonz commented Jun 18, 2024

@richiprosima please test_3 this

@EduPonz EduPonz merged commit b5c042f into master Jun 19, 2024
4 checks passed
@EduPonz EduPonz deleted the feature/cleanup_include_fastrtps branch June 19, 2024 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants