Skip to content

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Jan 21, 2026

Description

This is a minor user experience improvement.

I was reading the intra-process tutorial and saw that the table of contents isn't very useful:

image

In most cases, increasing the depth from 1 to 2 doesn't really "cost" anything and is pretty helpful IMO. Therefore, I looked for pages that aren't too short, have more than 1 level of sections, and use a ToC with a depth of 1, and increased their depth to 2.

image

Did you use Generative AI?

Additional Information

There's no reason not to try backporting this.


This is an automatic backport of pull request #6168 done by Mergify.

Signed-off-by: Christophe Bedard <bedard.christophe@gmail.com>
(cherry picked from commit 6eacc23)

# Conflicts:
#	source/Tutorials/Demos/Action-Introspection.rst
@mergify
Copy link
Contributor Author

mergify bot commented Jan 21, 2026

Cherry-pick of 6eacc23 has failed:

On branch mergify/bp/jazzy/pr-6168
Your branch is up to date with 'origin/jazzy'.

You are currently cherry-picking commit 6eacc23.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   source/How-To-Guides/Configure-ZeroCopy-loaned-messages.rst
	modified:   source/How-To-Guides/Installation-Troubleshooting.rst
	modified:   source/How-To-Guides/Launch-file-different-formats.rst
	modified:   source/How-To-Guides/Node-arguments.rst
	modified:   source/Tutorials/Demos/Intra-Process-Communication.rst

Unmerged paths:
  (use "git add/rm <file>..." as appropriate to mark resolution)
	deleted by us:   source/Tutorials/Demos/Action-Introspection.rst

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@mergify mergify bot added the conflicts label Jan 21, 2026
@christophebedard
Copy link
Member

We should just delete source/Tutorials/Demos/Action-Introspection.rst here.

Copy link
Collaborator

@fujitatomoya fujitatomoya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

action introspection is not available, so we need to remove that file.

Signed-off-by: Christophe Bedard <bedard.christophe@gmail.com>
@github-actions
Copy link

HTML artifacts: https://github.com/ros2/ros2_documentation/actions/runs/21192931919/artifacts/5197980810.

To view the resulting site:

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

@fujitatomoya fujitatomoya merged commit e700b00 into jazzy Jan 21, 2026
5 checks passed
@fujitatomoya fujitatomoya deleted the mergify/bp/jazzy/pr-6168 branch January 21, 2026 00:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants