|
| 1 | +<!-- Provide a general summary of your changes in the Title above --> |
| 2 | +<!-- It must be meaningful and coherent with the changes --> |
| 3 | + |
| 4 | +<!-- |
| 5 | + If this PR is still a Work in Progress [WIP], please open it as DRAFT. |
| 6 | + Please consider if any label should be added to this PR. |
| 7 | +--> |
| 8 | + |
| 9 | +## Description |
| 10 | +<!-- |
| 11 | + Describe changes in detail. |
| 12 | + This includes depicting the context, use case or current behavior and describe the proposed changes. |
| 13 | + If several features/bug fixes are included with these changes, please consider opening separated pull requests. |
| 14 | +--> |
| 15 | + |
| 16 | +<!-- |
| 17 | + In case of bug fixes, please provide the list of supported branches where this fix should be also merged. |
| 18 | + Please uncomment following line, adjusting the corresponding target branches for the backport. |
| 19 | +--> |
| 20 | +<!-- @Mergifyio backport 2.2.x 1.1.x 1.0.x --> |
| 21 | + |
| 22 | +<!-- If an issue is already opened, please uncomment next line with the corresponding issue number. --> |
| 23 | +<!-- Fixes #(issue) --> |
| 24 | + |
| 25 | +<!-- In case the changes are built over a previous pull request, please uncomment next line. --> |
| 26 | +<!-- This PR depends on #(PR) and must be merged after that one. --> |
| 27 | + |
| 28 | +## Contributor Checklist |
| 29 | + |
| 30 | +<!-- |
| 31 | + - If any of the elements of the following checklist is not applicable, substitute the checkbox [ ] by _N/A_: |
| 32 | + - If any of the elements of the following checklist is not fulfilled on purpose, please provide a reason and substitute the checkbox [ ] with ❌: or __NO__:. |
| 33 | +--> |
| 34 | + |
| 35 | +- [ ] Commit messages follow the project guidelines. <!-- External contributors should sign the DCO. Fast DDS developers must also refer to the internal Redmine task. --> |
| 36 | +- [ ] The code follows the style guidelines of this project. <!-- Please refer to the [Quality Declaration](https://github.com/eProsima/Fast-CDR/blob/master/QUALITY.md#linters-and-static-analysis-4v) for more information. --> |
| 37 | +- [ ] 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 <!-- Blackbox tests checking the new functionality are required. Changes that add/modify public API must include unit tests covering all possible cases. In case that no tests are provided, please justify why. --> |
| 38 | +- [ ] Any new/modified methods have been properly documented using Doxygen. <!-- Even internal classes, and private methods and members should be documented, not only the public API. --> |
| 39 | +- [ ] Changes are ABI compatible. <!-- Bug fixes should be ABI compatible if possible so a backport to previous affected releases can be made. --> |
| 40 | +- [ ] Changes are API compatible. <!-- Public API must not be broken within the same major release. --> |
| 41 | +- [ ] New feature has been added to the `versions.md` file (if applicable). |
| 42 | +- [ ] Applicable backports have been included in the description. |
| 43 | + |
| 44 | +## Reviewer Checklist |
| 45 | + |
| 46 | +- [ ] The PR has a milestone assigned. |
| 47 | +- [ ] The title and description correctly express the PR's purpose. |
| 48 | +- [ ] Check contributor checklist is correct. |
| 49 | +- [ ] Check CI results: changes do not issue any warning. |
| 50 | +- [ ] Check CI results: CI pass and failing tests are unrelated with the changes. |
0 commit comments