Skip to content

[20509] Update repository for Fast DDS 3.0.0 compatibility - #86

Merged
rsanchez15 merged 10 commits into
mainfrom
integration/bump-3.0.0
Sep 16, 2024
Merged

[20509] Update repository for Fast DDS 3.0.0 compatibility#86
rsanchez15 merged 10 commits into
mainfrom
integration/bump-3.0.0

Conversation

@LuciaEchevarria99

@LuciaEchevarria99 LuciaEchevarria99 commented May 20, 2024 •
edited by juanlofer-eprosima
Loading

Copy link
Copy Markdown
Contributor

@tempate tempate changed the title Update repository for Fast DDS 3.0.0 compatibility [20509] Update repository for Fast DDS 3.0.0 compatibility Jul 19, 2024
juanlofer-eprosima
juanlofer-eprosima requested changes Sep 2, 2024
View reviewed changes
Comment thread fastddsspy_participants/include/fastddsspy_participants/model/NetworkDatabase.hpp Outdated
Comment thread fastddsspy_participants/test/unittest/model/DataStreamerTest.cpp Outdated
Comment thread fastddsspy_tool/src/cpp/tool/Controller.cpp Outdated
LuciaEchevarria99 and others added 5 commits September 13, 2024 13:40
@LuciaEchevarria99
Update repository for Fast DDS 3.0.0 compatibility
a433477 
Signed-off-by: Lucia Echevarria <luciaechevarria@eprosima.com>
@LuciaEchevarria99
Regenerate types and change fastrtps dependency to fastdds
f6a86e1 
Signed-off-by: Lucia Echevarria <luciaechevarria@eprosima.com>
@LuciaEchevarria99
Continue updating to Fast DDS 3.x
9387a18 
Signed-off-by: Lucia Echevarria <luciaechevarria@eprosima.com>
@LuciaEchevarria99
Update to new changes in FastDDS
3f44213 
Signed-off-by: Lucia Echevarria <luciaechevarria@eprosima.com>
@juanlofer-eprosima @LuciaEchevarria99
Accept v6 interfaces and interface names in whitelist (#93)
98cd1bc 
Signed-off-by: Juan Lopez Fernandez <juanlopez@eprosima.com>
@LuciaEchevarria99
Remove tool_show_topic_dds test
c2e07b7 
Signed-off-by: Lucia Echevarria <luciaechevarria@eprosima.com>
@LuciaEchevarria99
Apply suggested changes
373bd9f 
Signed-off-by: Lucia Echevarria <luciaechevarria@eprosima.com>
juanlofer-eprosima
juanlofer-eprosima requested changes Sep 16, 2024
View reviewed changes

@juanlofer-eprosima juanlofer-eprosima left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update gtest version to 1.12 , and remove note about publish-types in monitoring configuration sections.

Comment thread fastddsspy_participants/src/cpp/model/DataStreamer.cpp Outdated
juanlofer-eprosima
juanlofer-eprosima requested changes Sep 16, 2024
View reviewed changes
Comment thread fastddsspy_tool/test/labels/XTSAN.list Outdated
@LuciaEchevarria99
Update log macro and fix XTSAN list
45c60d0 
Signed-off-by: Lucia Echevarria <luciaechevarria@eprosima.com>
@LuciaEchevarria99
Update documentation
d8a87ac 
Signed-off-by: Lucia Echevarria <luciaechevarria@eprosima.com>
@LuciaEchevarria99
Uncrustify
2463e94 
Signed-off-by: Lucia Echevarria <luciaechevarria@eprosima.com>
juanlofer-eprosima
juanlofer-eprosima approved these changes Sep 16, 2024
View reviewed changes

@juanlofer-eprosima juanlofer-eprosima left a comment •
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, tests passing -> https://github.com/eProsima/Fast-DDS-spy/actions/runs/10883479273

@rsanchez15
rsanchez15 merged commit f38df04 into main Sep 16, 2024
@rsanchez15
rsanchez15 deleted the integration/bump-3.0.0 branch September 16, 2024 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@juanlofer-eprosima juanlofer-eprosima juanlofer-eprosima approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@LuciaEchevarria99 @juanlofer-eprosima @rsanchez15