Skip to content

[DO NOT MERGE][ABI test] rolling / backport label removed by hand stays removed on the next push - #3383

Draft
fujitatomoya wants to merge 2 commits into
rollingfrom
abi-test/rolling-label-removed-by-hand
Draft

fujitatomoya wants to merge 2 commits into
rollingfrom
abi-test/rolling-label-removed-by-hand

Conversation

@fujitatomoya

Copy link
Copy Markdown
Collaborator

Case: target rolling, comment-only change. Two steps: after the first run has added the labels, a person removes backport-humble by hand and a second commit is pushed. The removed label must not come back.


This is a test pull request for the ABI checker (abi.yaml / abi-report.yaml, backed by
fujitatomoya/ros2-abi-action) and the
backport-<distro> candidate labels (#2555). It exists only for fujitatomoya to verify the
action result for one specific case, and it is closed without merging once that is done.
Maintainers: nothing to review here, please ignore this pull request.

🤖 Generated with Claude Code

…nd label case)

ABI checker / backport label test; this pull request is closed without merging.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@fujitatomoya

Copy link
Copy Markdown
Collaborator Author

Expected result

Step 1 (first commit, compatible)

  • abi check: success, label ABI compatible
  • backport labels added: backport-lyrical, backport-kilted, backport-jazzy, backport-humble

Step 2 (manual: fujitatomoya removes backport-humble by hand, then pushes one more comment-only commit)

  • abi check: success, label ABI compatible still present
  • backport-lyrical, backport-kilted, backport-jazzy still present
  • backport-humble not re-added (the unlabeled timeline event by a non-bot actor is honoured); the backport-label job logs Skipped label: backport-humble (removed by a maintainer)

…abel case)

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

ABI Compliance Check

✅ Verdict: compatible

Library Verdict Summary
libcomponent_manager.so ✅ compatible No ABI changes detected.
librclcpp.so ✅ compatible No ABI changes detected.
librclcpp_action.so ✅ compatible No ABI changes detected.
librclcpp_lifecycle.so ✅ compatible No ABI changes detected.
✅ libcomponent_manager.so — full abidiff report

Compared:

  • Base: lib-base/libcomponent_manager.so
  • Head: lib-pr/libcomponent_manager.so @ da868c9
(empty report — no differences printed by abidiff)
✅ librclcpp.so — full abidiff report

Compared:

  • Base: lib-base/librclcpp.so
  • Head: lib-pr/librclcpp.so @ da868c9
(empty report — no differences printed by abidiff)
✅ librclcpp_action.so — full abidiff report

Compared:

  • Base: lib-base/librclcpp_action.so
  • Head: lib-pr/librclcpp_action.so @ da868c9
(empty report — no differences printed by abidiff)
✅ librclcpp_lifecycle.so — full abidiff report

Compared:

  • Base: lib-base/librclcpp_lifecycle.so
  • Head: lib-pr/librclcpp_lifecycle.so @ da868c9
(empty report — no differences printed by abidiff)

Updated for commit da868c9 · check run · suppressions: /home/runner/work/_temp/ros2-abi-suppressions.txt

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.

1 participant