Skip to content

CI: update ros-tooling actions and remove the scheduled run - #57

Merged
fmrico merged 1 commit into
mainfrom
ci-fix
Sep 26, 2026
Merged

fmrico merged 1 commit into
mainfrom
ci-fix

Conversation

@fmrico

@fmrico fmrico commented Sep 26, 2026

Copy link
Copy Markdown
Member

The CI of main was disabled by GitHub after 60 days of inactivity, and its last runs failed because the old ros-tooling/setup-ros version uses an expired ROS apt key.

This PR:

  • updates ros-tooling/setup-ros to 0.7.19 and ros-tooling/action-ros-ci to 0.4.8 (still ROS 2 Humble on Ubuntu 22.04, as the main branch);
  • sets ROS_LOG_DIR, because the tests cannot create ~/.ros/log in the runner;
  • replaces the weekly scheduled run, which caused the automatic disabling, with a manual trigger.

The workflow has been re-enabled.

🤖 Generated with Claude Code

The old setup-ros version uses an expired ROS apt key, and the weekly
scheduled run made GitHub disable the workflow after 60 days of
inactivity.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings September 26, 2026 21:23

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

No unresolved review issues were identified.

Review effort: Lite
Findings: None

What changed in this PR

Updates the ROS 2 CI workflow for current tooling and manual execution.

Changes:

  • Updated ROS tooling actions.
  • Configured ROS_LOG_DIR.
  • Replaced scheduled runs with manual dispatch.
File Description
.github/​workflows/​main.yaml Updates CI actions, logging configuration, and triggers.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@fmrico
fmrico merged commit da44041 into main Sep 26, 2026
1 of 2 checks passed
@fmrico
fmrico deleted the ci-fix branch September 26, 2026 21:48
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