Skip to content

Conversation

@christophfroehlich
Copy link
Contributor

No description provided.

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 refactors GitHub Actions workflow files to use YAML anchors for eliminating duplicated event trigger configurations. The change introduces a &event anchor on the pull_request trigger and references it with *event for the push trigger, reducing code duplication across multiple workflow files.

Key changes:

  • Introduced YAML anchor &event on pull_request event configurations
  • Replaced duplicated push event configurations with *event references
  • Applied this pattern consistently across 18 GitHub Actions workflow files

Reviewed Changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
.github/workflows/rolling-semi-binary-build.yml Refactored to use YAML anchor for event triggers
.github/workflows/rolling-semi-binary-build-win.yml Refactored to use YAML anchor for event triggers
.github/workflows/rolling-coverage-build.yml Refactored to use YAML anchor for event triggers
.github/workflows/rolling-compatibility-kilted-binary-build.yml Refactored to use YAML anchor for event triggers
.github/workflows/rolling-compatibility-jazzy-binary-build.yml Refactored to use YAML anchor for event triggers
.github/workflows/rolling-compatibility-humble-binary-build.yml Refactored to use YAML anchor for event triggers
.github/workflows/rolling-binary-build.yml Refactored to use YAML anchor for event triggers
.github/workflows/kilted-semi-binary-build.yml Refactored to use YAML anchor for event triggers
.github/workflows/kilted-coverage-build.yml Refactored to use YAML anchor for event triggers
.github/workflows/kilted-binary-build.yml Refactored to use YAML anchor for event triggers
.github/workflows/jazzy-semi-binary-build.yml Refactored to use YAML anchor for event triggers
.github/workflows/jazzy-coverage-build.yml Refactored to use YAML anchor for event triggers
.github/workflows/jazzy-binary-build.yml Refactored to use YAML anchor for event triggers
.github/workflows/humble-semi-binary-build.yml Refactored to use YAML anchor for event triggers
.github/workflows/humble-coverage-build.yml Refactored to use YAML anchor for event triggers
.github/workflows/humble-binary-build.yml Refactored to use YAML anchor for event triggers

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

@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
@codecov
Copy link

codecov bot commented Oct 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.13%. Comparing base (0ac3475) to head (bd3eaf5).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1953   +/-   ##
=======================================
  Coverage   85.13%   85.13%           
=======================================
  Files         143      143           
  Lines       13742    13742           
  Branches     1200     1200           
=======================================
  Hits        11699    11699           
  Misses       1636     1636           
  Partials      407      407           
Flag Coverage Δ
unittests 85.13% <ø> (ø)

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.

@christophfroehlich christophfroehlich merged commit 34c5ff3 into master Oct 6, 2025
15 checks passed
@christophfroehlich christophfroehlich deleted the ci/yaml branch October 6, 2025 06:17
mergify bot pushed a commit that referenced this pull request Oct 6, 2025
(cherry picked from commit 34c5ff3)
mergify bot pushed a commit that referenced this pull request Oct 6, 2025
(cherry picked from commit 34c5ff3)
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