Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[21053] Move DR TypeConsistencyEnforcement & DataRepresentation from TypeConsistency to DataReaderQos #779

Merged
merged 2 commits into from
May 27, 2024

Conversation

JesusPoderoso
Copy link
Contributor

@JesusPoderoso JesusPoderoso commented May 21, 2024

Description

This PR updates and improves the documentation of TypeConsistency and DataRepresentation QoS policies

Contributor Checklist

  • Commit messages follow the project guidelines.
  • 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.

…entationQosPolicy to correct place

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Copy link
Member

@Mario-DL Mario-DL left a comment

Choose a reason for hiding this comment

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

We would need to change the references to eprosima::fastdds::dds::TypeConsistencyQos to eprosima::fastdds::dds::TypeConsistencyEnforcementQos, for instance, in the docs/fastdds/api_reference/dds_pim/subscriber/typeconsistencyqos.rst.
Also in the DDSCodeTester there are some declarations consistency_qos.representation that should be changed to representation()

@JesusPoderoso JesusPoderoso changed the title [21053] Move DR TypeConsistencyEnforcementQosPolicy & DataRepresentationQosPolicy… [21053] Move DR TypeConsistencyEnforcement & DataRepresentation from TypeConsistency to DataReaderQos May 23, 2024
@JesusPoderoso JesusPoderoso force-pushed the feature/21053 branch 4 times, most recently from 68003e0 to 84c1e90 Compare May 23, 2024 14:27
Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
@JesusPoderoso
Copy link
Contributor Author

@richiprosima please test_3 this

Copy link
Member

@Mario-DL Mario-DL left a comment

Choose a reason for hiding this comment

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

LGTM

@EduPonz EduPonz merged commit 4119d79 into master May 27, 2024
4 checks passed
@EduPonz EduPonz deleted the feature/21053 branch May 27, 2024 07:51
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants