Skip to content

robostack-bot: pinning update commits carry [ci skip] - #121

Merged
Tobias-Fischer merged 2 commits into
mainfrom
bot-pinning-ci-skip
Oct 21, 2026
Merged

Tobias-Fischer merged 2 commits into
mainfrom
bot-pinning-ci-skip

Conversation

@Tobias-Fischer

Copy link
Copy Markdown
Collaborator

The bot's daily update-conda-forge-pinning run re-creates its commit on top of main and force-pushes it (that's how peter-evans/create-pull-request updates a PR), so the pinning PR (#94) restarts its build every day, and a pinning update can rebuild a lot.

Now the commit message of pinning updates ends in [ci skip] (the PR title stays the same), so those pushes don't start CI. To build such a PR, push any commit without the marker, e.g. git commit --allow-empty -m "Build". The README's bot table says so too. Other bot commands are unchanged.

The schedule runs the default branch's bot.yaml, so this takes effect once merged; until then the next daily run will push #94 again and start a build.

🤖 Generated with Claude Code

create-pull-request re-creates the bot's commit on top of main on every
run and force-pushes it, so the daily update-conda-forge-pinning run
restarted the PR's build (which can rebuild a lot) every day. Its commits
now carry [ci skip]; to build such a PR, push a commit without the marker.
The PR title stays as before.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Tobias-Fischer

Copy link
Copy Markdown
Collaborator Author

This is ready for review :)

Empty commit without the skip marker, so the pull request's checks run before merging.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

ABI check (linux-64)

rolling linux-64: **1 need their dependents rebuilt**
package released → PR verdict details
rplidar-ros 2.1.0 (build 20) → 2.1.4 incompatible: removed or changed symbols: rebuild the dependents removed: librplidar_composition_node.so
rebuild 0 dependents (of 0 released dependents, these link to it): none
2 headers changed (check for layout/inline changes): -rplidar_ros/rplidar_node.hpp, -rplidar_ros/visibility.h
pins changed: numpy >=1.23,<3 → >=1.25,<3

Compared with the newest builds on https://repo.prefix.dev/tobias-fischer/robostack-distro-test/linux-64 and https://repo.prefix.dev/robostack-rolling/linux-64.

About this check

Each package this pull request built, compared with its newest released build (tools/abi_check.py, libabigail):

  • compatible / additions: the package can be bumped on its own;
  • soname / incompatible: the packages that depend on it need a rebuild (build number bump) in the same pull request.

Exported symbols are compared (the released packages are stripped, so struct layouts aren't); unchanged installed headers rule out layout and inline changes, changed ones are listed for review. Dependents to rebuild are the released ones linking to a changed library. A changed pin is part of the comparison.

@Tobias-Fischer
Tobias-Fischer merged commit 3c57280 into main Oct 21, 2026
30 checks passed
@Tobias-Fischer
Tobias-Fischer deleted the bot-pinning-ci-skip branch October 21, 2026 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants