Skip to content

[22768] Set index on exception members #172

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
MiguelCompany merged 1 commit into from
Feb 19, 2025
Merged

[22768] Set index on exception members #172

MiguelCompany merged 1 commit into from
Feb 19, 2025

Conversation

@MiguelCompany
Copy link
Member

@MiguelCompany MiguelCompany commented Feb 19, 2025 •
edited by richiware
Loading

Description

This PR makes exception members have a consistent index attribute.

@Mergifyio backport 4.0.x

Contributor Checklist

  • Commit messages follow the project guidelines.
  • 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
  • N/A: Any new/modified methods have been properly documented.
  • Changes are backport compatible: they do NOT break ABI nor change library core behavior.
  • Changes are API compatible.
  • 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.

@MiguelCompany
Refs #22768. Set index on exception members.
295df78 
Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
@MiguelCompany MiguelCompany added this to the v4.1.0 milestone Feb 19, 2025
@MiguelCompany MiguelCompany mentioned this pull request Feb 19, 2025
Merged
8 tasks
@MiguelCompany
Copy link
Member Author

MiguelCompany commented Feb 19, 2025

@Mergifyio backport 4.0.x

@mergify
Copy link

mergify bot commented Feb 19, 2025 •
edited
Loading

backport 4.0.x

✅ Backports have been created

Details

@MiguelCompany MiguelCompany merged commit ecdfd12 into master Feb 19, 2025
2 checks passed
@MiguelCompany MiguelCompany deleted the feature/rpc/22768 branch February 19, 2025 09:27
mergify bot pushed a commit that referenced this pull request Feb 19, 2025
@MiguelCompany @mergify
Set index on exception members (#172)
bc2c728 
Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
(cherry picked from commit ecdfd12)
@mergify mergify bot mentioned this pull request Feb 19, 2025
Merged
8 tasks
MiguelCompany added a commit that referenced this pull request Feb 19, 2025
@mergify @MiguelCompany
Set index on exception members (#172) (#173)
6cb0b57 
Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
(cherry picked from commit ecdfd12)

Co-authored-by: Miguel Company <miguelcompany@eprosima.com>
@MiguelCompany MiguelCompany mentioned this pull request May 8, 2025
Merged
8 tasks
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

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v4.1.0

Development

Successfully merging this pull request may close these issues.

3 participants

@MiguelCompany @richiware