Skip to content

[24067] Refactor RPC documentation#1211

Merged
emiliocuestaf merged 23 commits intomasterfrom
fast/rpc-refactor
Feb 25, 2026
Merged

[24067] Refactor RPC documentation#1211
emiliocuestaf merged 23 commits intomasterfrom
fast/rpc-refactor

Conversation

@emiliocuestaf
Copy link
Contributor

@emiliocuestaf emiliocuestaf commented Feb 13, 2026 •
edited
Loading

Description

This PR updates RPC documentation to reflect what it is actually implemented in Fast DDS

This PR depends on #1213 and must be merged after that one.

Contributor Checklist

  • Commit messages follow the project guidelines.
  • [N/A] Code snippets related to the added documentation have been provided.
  • Documentation tests pass locally.
  • The Pro version badge has been added if the documented feature is exclusive to Fast DDS Pro.
  • [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.

@emiliocuestaf emiliocuestaf requested review from richiprosima and removed request for richiprosima February 17, 2026 06:58
@emiliocuestaf emiliocuestaf changed the title [24067] Refactor RPC docu [24067] Refactor RPC documentation Feb 17, 2026
@emiliocuestaf emiliocuestaf requested review from richiprosima and removed request for richiprosima February 17, 2026 10:15
@MiguelCompany MiguelCompany mentioned this pull request Feb 17, 2026
Merged
11 tasks
@MiguelCompany MiguelCompany added this to the v3.5.0 milestone Feb 17, 2026
@emiliocuestaf emiliocuestaf requested review from richiprosima and removed request for richiprosima February 17, 2026 11:52
MiguelCompany
MiguelCompany previously requested changes Feb 17, 2026
View reviewed changes
@emiliocuestaf emiliocuestaf requested review from richiprosima and removed request for richiprosima February 17, 2026 13:31
@emiliocuestaf emiliocuestaf changed the base branch from master to ci/doxygen February 19, 2026 11:25
@emiliocuestaf emiliocuestaf requested review from richiprosima and removed request for richiprosima February 20, 2026 06:18
@emiliocuestaf emiliocuestaf requested review from richiprosima and removed request for richiprosima February 23, 2026 14:38
@emiliocuestaf
Remove unnecessary alias file
6f63409 
Signed-off-by: Emilio Cuesta <emiliocuesta@eprosima.com>
@emiliocuestaf
Remove ignoring D000
7a45606 
Signed-off-by: Emilio Cuesta <emiliocuesta@eprosima.com>
@emiliocuestaf
Remove duplicated refs
5cb1b6a 
Signed-off-by: Emilio Cuesta <emiliocuesta@eprosima.com>
@emiliocuestaf
Add text to references
8adc6f0 
Signed-off-by: Emilio Cuesta <emiliocuesta@eprosima.com>
@emiliocuestaf
Move conflictive refs to separate file
e781cdc 
Signed-off-by: Emilio Cuesta <emiliocuesta@eprosima.com>
@emiliocuestaf
Resolve missing refs
8055993 
Signed-off-by: Emilio Cuesta <emiliocuesta@eprosima.com>
@emiliocuestaf
Add ref.any to ignored warnings
1089cc2 
Signed-off-by: Emilio Cuesta <emiliocuesta@eprosima.com>
@emiliocuestaf
Remove the flag to ignore any as its not required anymore
4fadfd0 
Signed-off-by: Emilio Cuesta <emiliocuesta@eprosima.com>
@emiliocuestaf
Add set of references that can fail
cd1b61b 
Signed-off-by: Emilio Cuesta <emiliocuesta@eprosima.com>
@emiliocuestaf
Refactor RPC docu
f1340e9 
Signed-off-by: Emilio Cuesta <emiliocuesta@eprosima.com>
@emiliocuestaf
Check RPC code/examples only on pro
f9f3521 
Signed-off-by: Emilio Cuesta <emiliocuesta@eprosima.com>
@emiliocuestaf
Move RPC from DDSCodeTests.cpp to ProDDSCodeTester.cpp
f2226e2 
Signed-off-by: Emilio Cuesta <emiliocuesta@eprosima.com>
@emiliocuestaf
Partial review
9efaa3f 
Signed-off-by: Emilio Cuesta <emiliocuesta@eprosima.com>
@emiliocuestaf
Replace DDSCodeTester by ProDDSCodeTester when required
6419c01 
Signed-off-by: Emilio Cuesta <emiliocuesta@eprosima.com>
@emiliocuestaf
Some missing comments
22a3f9c 
Signed-off-by: Emilio Cuesta <emiliocuesta@eprosima.com>
@emiliocuestaf
Remove uneeded headers
15b03cc 
Signed-off-by: Emilio Cuesta <emiliocuesta@eprosima.com>
@emiliocuestaf
Remove redundant PRO badge
12eba64 
Signed-off-by: Emilio Cuesta <emiliocuesta@eprosima.com>
@emiliocuestaf emiliocuestaf requested review from richiprosima and removed request for richiprosima February 25, 2026 10:22
richiware
richiware requested changes Feb 25, 2026
View reviewed changes
@emiliocuestaf
Apply revision
20d5bad 
Signed-off-by: Emilio Cuesta <emiliocuesta@eprosima.com>
@richiware richiware requested review from MiguelCompany and removed request for MiguelCompany February 25, 2026 12:32
@emiliocuestaf emiliocuestaf merged commit 6ed88c3 into master Feb 25, 2026
5 checks passed
@emiliocuestaf emiliocuestaf deleted the fast/rpc-refactor branch February 25, 2026 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@richiware richiware richiware approved these changes

@MiguelCompany MiguelCompany MiguelCompany left review comments

@richiprosima richiprosima Awaiting requested review from richiprosima

Assignees

No one assigned

Labels

ci-pending fast dds pro ready-to-merge

Projects

None yet

Milestone

v3.5.0 - Frozen ❄️

Development

Successfully merging this pull request may close these issues.

3 participants

@emiliocuestaf @richiware @MiguelCompany