Skip to content

[23325] Add generic interfaces for RPC servers #241

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
Merged
merged 2 commits into from
Jun 17, 2025
Merged

[23325] Add generic interfaces for RPC servers #241

merged 2 commits into from
Jun 17, 2025

Conversation

MiguelCompany
Copy link
Member

@MiguelCompany MiguelCompany commented Jun 13, 2025 •
edited
Loading

Description

Expose the server and request interfaces from eProsima/Fast-DDS#5873

Contributor Checklist

  • Commit messages follow the project guidelines.
  • N/A: Tests that thoroughly check the new feature have been added/Regression tests checking the bug and its fix have been added; the added tests pass locally
  • Changes are API compatible.
  • 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.
  • Check CI results: changes do not issue any warning.
  • Check CI results: failing tests are unrelated with the changes.

@MiguelCompany
Refs #23325. Update swig file for RPC interfaces.
664b7f9 
Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
@MiguelCompany MiguelCompany added this to the v2.3.0 milestone Jun 13, 2025
@MiguelCompany MiguelCompany mentioned this pull request Jun 13, 2025
Merged
7 tasks
Carlosespicur
Carlosespicur previously approved these changes Jun 16, 2025
View reviewed changes
@Carlosespicur Carlosespicur self-requested a review June 16, 2025 09:17
@github-actions github-actions bot added the ci-pending PR which CI is running label Jun 16, 2025
@MiguelCompany
Refs #23325. Update generated code.
5a59e8b 
Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
@MiguelCompany MiguelCompany requested review from Carlosespicur and removed request for Carlosespicur June 17, 2025 13:04
@MiguelCompany MiguelCompany merged commit d294198 into main Jun 17, 2025
6 of 8 checks passed
@MiguelCompany MiguelCompany deleted the feature/23325 branch June 17, 2025 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Carlosespicur Carlosespicur Carlosespicur approved these changes

Assignees

No one assigned

Labels

ci-pending PR which CI is running

Projects

None yet

Milestone

v2.3.0 - Frozen ❄️

Development

Successfully merging this pull request may close these issues.

2 participants

@MiguelCompany @Carlosespicur