Skip to content

Conversation

@christophebedard
Copy link
Member

Description

Relates to #5820.

I noticed that there are asterisks in the field types tables (especially for Python types), but there's no corresponding note: https://docs.ros.org/en/rolling/Concepts/Basic/About-Interfaces.html#field-types.

I dug through the git history, and found that the note is the sentence below the 2nd table:

All types that are more permissive than their ROS definition enforce the ROS constraints in range and length by software

The asterisk was simply removed because it was not valid RST: 2aafc06. Restore it while making sure it's valid RST.

Did you use Generative AI?

Additional Information

@christophebedard christophebedard added the backport-all backport at reviewers discretion; from rolling to all versions label Oct 22, 2025
@github-actions
Copy link

HTML artifacts: https://github.com/ros2/ros2_documentation/actions/runs/18725369426/artifacts/4343471353.

To view the resulting site:

  1. Click on the above link to download the artifacts archive
  2. Extract it
  3. Open html-artifacts-5978/index.html in your favorite browser

@christophebedard
Copy link
Member Author

@InvincibleRMC I noticed this while looking at this page for #5820. I think this sentence isn't very clear unless you've looked at the rosidl_generator_py code, though.

@christophebedard christophebedard merged commit 8aeaf2a into rolling Oct 23, 2025
5 checks passed
@christophebedard christophebedard deleted the christophebedard/about-interfaces-restore-asterisk branch October 23, 2025 17:32
mergify bot pushed a commit that referenced this pull request Oct 23, 2025
Signed-off-by: Christophe Bedard <[email protected]>
(cherry picked from commit 8aeaf2a)
mergify bot pushed a commit that referenced this pull request Oct 23, 2025
Signed-off-by: Christophe Bedard <[email protected]>
(cherry picked from commit 8aeaf2a)
mergify bot pushed a commit that referenced this pull request Oct 23, 2025
Signed-off-by: Christophe Bedard <[email protected]>
(cherry picked from commit 8aeaf2a)
christophebedard added a commit that referenced this pull request Oct 23, 2025
(cherry picked from commit 8aeaf2a)

Signed-off-by: Christophe Bedard <[email protected]>
Co-authored-by: Christophe Bedard <[email protected]>
christophebedard added a commit that referenced this pull request Oct 23, 2025
(cherry picked from commit 8aeaf2a)

Signed-off-by: Christophe Bedard <[email protected]>
Co-authored-by: Christophe Bedard <[email protected]>
christophebedard added a commit that referenced this pull request Oct 23, 2025
(cherry picked from commit 8aeaf2a)

Signed-off-by: Christophe Bedard <[email protected]>
Co-authored-by: Christophe Bedard <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-all backport at reviewers discretion; from rolling to all versions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants