Skip to content

Conversation

christophfroehlich
Copy link
Contributor

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 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 Actions workflows to optimize CI/CD execution by only triggering builds when relevant files are modified. The filters include source code files (cpp, hpp, h, py), configuration files (yaml, xml, CMakeLists.txt, package.xml), repository files, and the workflow files themselves.

  • Added consistent path filtering across all workflow files
  • Configured filters for both pull_request and push triggers where applicable
  • Included relevant repository files specific to each workflow's branch/version

Reviewed Changes

Copilot reviewed 26 out of 26 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
.github/workflows/rolling-semi-binary-build.yml Added path filters for rolling branch semi-binary builds
.github/workflows/rolling-semi-binary-build-win.yml Added path filters for rolling branch Windows builds
.github/workflows/rolling-rhel-semi-binary-build.yml Added path filters for rolling branch RHEL builds
.github/workflows/rolling-debian-build.yml Added path filters for rolling branch Debian builds
.github/workflows/rolling-compatibility-build.yml Added path filters for rolling branch compatibility builds
.github/workflows/rolling-build-source.yml Added path filters for rolling branch source builds
.github/workflows/rolling-build-downstream.yml Added path filters for rolling branch downstream builds
.github/workflows/rolling-build-coverage.yml Added path filters for rolling branch coverage builds
.github/workflows/rolling-binary-build.yml Added path filters for rolling branch binary builds
.github/workflows/rolling-abi-compatibility.yml Added path filters for rolling branch ABI compatibility checks
.github/workflows/jazzy-semi-binary-build.yml Added path filters for jazzy branch semi-binary builds
.github/workflows/jazzy-rhel-semi-binary-build.yml Added path filters for jazzy branch RHEL builds
.github/workflows/jazzy-debian-build.yml Added path filters for jazzy branch Debian builds
.github/workflows/jazzy-build-source.yml Added path filters for jazzy branch source builds
.github/workflows/jazzy-build-downstream.yml Added path filters for jazzy branch downstream builds
.github/workflows/jazzy-build-coverage.yml Added path filters for jazzy branch coverage builds
.github/workflows/jazzy-binary-build.yml Added path filters for jazzy branch binary builds
.github/workflows/jazzy-abi-compatibility.yml Added path filters for jazzy branch ABI compatibility checks
.github/workflows/humble-semi-binary-build.yml Added path filters for humble branch semi-binary builds
.github/workflows/humble-rhel-semi-binary-build.yml Added path filters for humble branch RHEL builds
.github/workflows/humble-debian-build.yml Added path filters for humble branch Debian builds
.github/workflows/humble-build-source.yml Added path filters for humble branch source builds
.github/workflows/humble-build-downstream.yml Added path filters for humble branch downstream builds
.github/workflows/humble-build-coverage.yml Added path filters for humble branch coverage builds
.github/workflows/humble-binary-build.yml Added path filters for humble branch binary builds
.github/workflows/humble-abi-compatibility.yml Added path filters for humble branch ABI compatibility checks

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

@codecov-commenter
Copy link

codecov-commenter commented Oct 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.35%. Comparing base (487d2e9) to head (891fe23).

Additional details and impacted files
@@             Coverage Diff              @@
##           ros2-master     #503   +/-   ##
============================================
  Coverage        82.35%   82.35%           
============================================
  Files               29       29           
  Lines             1984     1984           
  Branches           114      114           
============================================
  Hits              1634     1634           
  Misses             281      281           
  Partials            69       69           
Flag Coverage Δ
unittests 82.35% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link

@Copilot 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 26 out of 26 changed files in this pull request and generated 3 comments.


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

@christophfroehlich christophfroehlich merged commit c0cdb15 into ros2-master Oct 6, 2025
27 checks passed
@christophfroehlich christophfroehlich deleted the ci/paths-filter branch October 6, 2025 06:22
mergify bot pushed a commit that referenced this pull request Oct 6, 2025
(cherry picked from commit c0cdb15)
mergify bot pushed a commit that referenced this pull request Oct 6, 2025
(cherry picked from commit c0cdb15)
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.

3 participants