Skip to content

3di | Triaging and reporting an issue - #6901

Merged
tfoote merged 18 commits into
ros2:rollingfrom
3di-for-openrobotics:triaging-reporting-an-issue
Jul 31, 2026
Merged

3di | Triaging and reporting an issue#6901
tfoote merged 18 commits into
ros2:rollingfrom
3di-for-openrobotics:triaging-reporting-an-issue

Conversation

@keithkirkwood-3di

@keithkirkwood-3di keithkirkwood-3di commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Description

Refactor/consolidation and partial rewrite of content from several sources to create new articles for Triaging an issue, and Reporting an issue under Contributing section.

Related to: #6828

Did you use Generative AI?

No

Additional Information

Documentation updates from 3di Information Solutions, agreed with Geoff and Tully.

@keithkirkwood-3di

Copy link
Copy Markdown
Contributor Author

@kscottz @tfoote Hello - I've had an odd linter failure in this PR. Doesn't look like anything to do with my updates (I think). Could you advise on what I should do here?

Comment thread source/The-ROS2-Project/Contributing/Contributing-to-code/Reporting-an-issue.rst Outdated

@fujitatomoya fujitatomoya left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

i have a couple of minor comments on this PR.

i do understand that this PR is trying to restructure the tree but it would be nice to fix at the same time. btw, my comments should not block this PR.

Comment thread source/The-ROS2-Project/Contributing/Contributing-to-code/Reporting-an-issue.rst Outdated
Comment thread source/The-ROS2-Project/Contributing/Contributing-to-code/Reporting-an-issue.rst Outdated

@kscottz kscottz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This is a good start but there's probably some more context that we could provide to make it better. Since this is geared towards new users there's a couple of things that we're failing to mention / cross reference in the documentation, including:

  • How to figure out what ROS package is causing the issue and where to find that package's source code repository.
  • How to use ros2 doctor to report your system configuration.
  • As @fujitatomoya mentioned, how to use the report template.
  • Where to find the appropriate log files in ROS.

Comment thread source/The-ROS2-Project/Contributing/Contributing-to-code/Reporting-an-issue.rst Outdated
Comment thread source/The-ROS2-Project/Contributing/Contributing-to-code/Reporting-an-issue.rst Outdated
Comment thread source/The-ROS2-Project/Contributing/Contributing-to-code/Triaging-an-issue.rst Outdated
Comment thread source/The-ROS2-Project/Contributing/Contributing-to-code/Reporting-an-issue.rst Outdated
Comment thread source/The-ROS2-Project/Contributing/Contributing-to-code/Reporting-an-issue.rst Outdated
Comment thread source/The-ROS2-Project/Contributing/Contributing-to-code/Triaging-an-issue.rst Outdated
Comment thread source/The-ROS2-Project/Contributing/Contributing-to-code/Triaging-an-issue.rst Outdated
gbiggs and others added 4 commits July 17, 2026 15:07
Signed-off-by: Geoffrey Biggs <gbiggs@killbots.net>
Signed-off-by: Geoffrey Biggs <gbiggs@killbots.net>
Signed-off-by: Geoffrey Biggs <gbiggs@killbots.net>
@tfoote
tfoote dismissed vmbrasseur’s stale review July 30, 2026 19:37

Comments resolved and Kat's reviewed after. Keeping things moving we can iterate on it again.

@tfoote
tfoote enabled auto-merge (squash) July 30, 2026 19:38
@tfoote
tfoote disabled auto-merge July 31, 2026 07:29
@tfoote
tfoote enabled auto-merge (squash) July 31, 2026 07:30
@tfoote

tfoote commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

@mergify queue

@mergify

mergify Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Merge Queue Status

  • 🟠 Waiting for queue conditions
  • ⏳ Enter queue
  • ⏳ Run checks
  • ⏳ Merge
Waiting for
  • -closed [📌 queue requirement]
All conditions
  • -closed [📌 queue requirement]
  • -conflict [📌 queue requirement]
  • -draft [📌 queue requirement]
  • any of [📌 queue -> configuration change requirements]:
    • -mergify-configuration-changed
    • check-success = Configuration changed
  • any of [🔀 queue conditions]:
    • all of [📌 queue conditions of queue rule default]:
      • #review-threads-unresolved = 0 [🛡 GitHub branch protection]
      • github-require-last-push-approval [🛡 GitHub branch protection]
      • github-review-approved [🛡 GitHub branch protection]
      • any of [🛡 GitHub branch protection]:
        • check-success = build
        • check-neutral = build
        • check-skipped = build
      • any of [🛡 GitHub branch protection]:
        • check-success = lint
        • check-neutral = lint
        • check-skipped = lint
      • any of [🛡 GitHub branch protection]:
        • check-success = test
        • check-neutral = test
        • check-skipped = test

@tfoote
tfoote merged commit bf9db74 into ros2:rolling Jul 31, 2026
7 checks passed
@tfoote

tfoote commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

@mergify backport lyrical

@mergify

mergify Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

backport lyrical

❌ No backport have been created

Details
  • Backport to branch lyrical failed

Git reported the following error:

To https://github.com/ros2/ros2_documentation
 ! [rejected]        mergify/bp/lyrical/pr-6901 -> mergify/bp/lyrical/pr-6901 (fetch first)
error: failed to push some refs to 'https://github.com/ros2/ros2_documentation'
hint: Updates were rejected because the remote contains work that you do not
hint: have locally. This is usually caused by another repository pushing to
hint: the same ref. If you want to integrate the remote changes, use
hint: 'git pull' before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.

@fujitatomoya

Copy link
Copy Markdown
Collaborator

@keithkirkwood-3di can you resolve the conflicts in the backport PRs? or are we not backporting those fixes to downstream branches?

@fujitatomoya

Copy link
Copy Markdown
Collaborator

@keithkirkwood-3di can you resolve the conflicts in the backport PRs? or are we not backporting those fixes to downstream branches?

i guess we are already doing this, see #7031

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.

6 participants