Skip to content

Conversation

@christophfroehlich
Copy link
Member

as we have in ros2_control repo

@christophfroehlich christophfroehlich added backport-humble Triggers PR backport to ROS 2 humble. backport-jazzy Triggers PR backport to ROS 2 jazzy. labels Oct 5, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds path filters to GitHub workflow files to optimize CI/CD execution by only running workflows when relevant files are modified. The changes implement filters similar to those used in the ros2_control repository.

Key changes:

  • Added path filters to all GitHub workflow files for both pull_request and push triggers
  • Configured filters to trigger on ROS-specific files (.action, .msg, .srv), build files (CMakeLists.txt, package.xml), and the workflow files themselves
  • Added repository-specific .repos files to downstream build workflows

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.github/workflows/rolling-build.yml Added path filters for pull_request and push triggers
.github/workflows/rolling-build-win.yml Added path filters for pull_request and push triggers
.github/workflows/rolling-binary-downstream-build.yml Added path filters for pull_request trigger including repos files
.github/workflows/jazzy-build.yml Added path filters for pull_request and push triggers
.github/workflows/jazzy-binary-downstream-build.yml Added path filters for pull_request trigger including repos files
.github/workflows/humble-build.yml Added path filters for pull_request and push triggers
.github/workflows/humble-binary-downstream-build.yml Added path filters for pull_request trigger including repos files

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@christophfroehlich christophfroehlich merged commit 2ca19b7 into master Oct 6, 2025
12 checks passed
@christophfroehlich christophfroehlich deleted the ci/paths-filter branch October 6, 2025 06:37
mergify bot pushed a commit that referenced this pull request Oct 6, 2025
(cherry picked from commit 2ca19b7)
mergify bot pushed a commit that referenced this pull request Oct 6, 2025
(cherry picked from commit 2ca19b7)
christophfroehlich pushed a commit that referenced this pull request Oct 6, 2025
christophfroehlich pushed a commit that referenced this pull request Oct 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-humble Triggers PR backport to ROS 2 humble. backport-jazzy Triggers PR backport to ROS 2 jazzy.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants