Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CI] Add jazzy and other minor updates #1521

Merged
merged 12 commits into from
May 8, 2024
Merged

[CI] Add jazzy and other minor updates #1521

merged 12 commits into from
May 8, 2024

Conversation

christophfroehlich
Copy link
Contributor

@christophfroehlich christophfroehlich commented Apr 29, 2024

  • Add Jazzy workflows (I decided to create dedicated files, because it easier to maintain if we branch for jazzy)
  • Add filters to run the ROS - workflows only if code has changed, and the doc workflow only if docs have changed. (branches+path filter have to match both to run the workflow)
  • Check Docs - workflow now uses the correct branch of control.ros.org (not every page is available on every distro any more).

@christophfroehlich christophfroehlich added backport-humble This label should be used by maintaines only! Label triggers PR backport to ROS2 humble. backport-iron This label should be used by maintaines only! Label triggers PR backport to ROS2 Iron. labels Apr 29, 2024
Copy link

codecov bot commented Apr 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.53%. Comparing base (607755e) to head (e53071f).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1521   +/-   ##
=======================================
  Coverage   88.53%   88.53%           
=======================================
  Files         101      101           
  Lines        8502     8502           
  Branches      759      759           
=======================================
  Hits         7527     7527           
  Misses        706      706           
  Partials      269      269           
Flag Coverage Δ
unittests 88.53% <ø> (ø)

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

Copy link
Contributor

@fmauch fmauch left a comment

Choose a reason for hiding this comment

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

Without ros-controls/ros2_control_ci#72 changes seem find, hence approving.

However, it probably makes sense to incorporate that into this PR, as well.

.github/workflows/jazzy-coverage-build.yml Outdated Show resolved Hide resolved
@christophfroehlich christophfroehlich merged commit 736d379 into master May 8, 2024
18 of 21 checks passed
@christophfroehlich christophfroehlich deleted the ci_jazzy branch May 8, 2024 21:56
mergify bot pushed a commit that referenced this pull request May 8, 2024
* Add file type filters to workflows

* Add jazzy jobs and update readmes

* Checkout correct branch for docs

* Deactivate jazzy - coverage build

* Specify workflow names

* Update upstream pre-commit wf

* Update coverage build inputs

---------

Co-authored-by: Bence Magyar <bence.magyar.robotics@gmail.com>
(cherry picked from commit 736d379)

# Conflicts:
#	.github/workflows/rolling-abi-compatibility.yml
mergify bot pushed a commit that referenced this pull request May 8, 2024
* Add file type filters to workflows

* Add jazzy jobs and update readmes

* Checkout correct branch for docs

* Deactivate jazzy - coverage build

* Specify workflow names

* Update upstream pre-commit wf

* Update coverage build inputs

---------

Co-authored-by: Bence Magyar <bence.magyar.robotics@gmail.com>
(cherry picked from commit 736d379)

# Conflicts:
#	.github/workflows/iron-abi-compatibility.yml
christophfroehlich added a commit that referenced this pull request May 8, 2024
* Add file type filters to workflows

* Add jazzy jobs and update readmes

* Checkout correct branch for docs

* Deactivate jazzy - coverage build

* Specify workflow names

* Update upstream pre-commit wf

* Update coverage build inputs

---------

Co-authored-by: Bence Magyar <bence.magyar.robotics@gmail.com>
(cherry picked from commit 736d379)

# Conflicts:
#	.github/workflows/iron-abi-compatibility.yml

Co-authored-by: Christoph Fröhlich <christophfroehlich@users.noreply.github.com>
christophfroehlich added a commit that referenced this pull request May 8, 2024
* Add file type filters to workflows

* Add jazzy jobs and update readmes

* Checkout correct branch for docs

* Deactivate jazzy - coverage build

* Specify workflow names

* Update upstream pre-commit wf

* Update coverage build inputs

---------

Co-authored-by: Bence Magyar <bence.magyar.robotics@gmail.com>
(cherry picked from commit 736d379)

# Conflicts:
#	.github/workflows/rolling-abi-compatibility.yml

Co-authored-by: Christoph Fröhlich <christophfroehlich@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-humble This label should be used by maintaines only! Label triggers PR backport to ROS2 humble. backport-iron This label should be used by maintaines only! Label triggers PR backport to ROS2 Iron.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants