Skip to content

Update CI (backport #71) - #72

Merged
David-LP99 merged 2 commits into
kiltedfrom
mergify/bp/kilted/pr-71
Sep 14, 2026
Merged

David-LP99 merged 2 commits into
kiltedfrom
mergify/bp/kilted/pr-71

Conversation

@mergify

@mergify mergify Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Description

This PR aims to update the CI structure for the repository.

Main changes

  • Remove coverage from nightly.yaml
  • Update docker images and distributions
  • Add lyrical in the matrix of the nightly CI
  • Bump version of the CI actions
    This is an automatic backport of pull request Update CI #71 done by Mergify.

* Bump CI actions

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>

* Fix codecov

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>

* Fix codecov 2

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>

* Remove coverage from nightly

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>

* Update CI

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>

* Set 'rolling' to 'ubuntu:24'

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>

* Fix coverage 3

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>

* Fix coverage 4

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>

* Clean debugging options

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>

---------

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
(cherry picked from commit 383fc6d)

# Conflicts:
#	.github/workflows/nightly.yml
@mergify

mergify Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor Author

Cherry-pick of 383fc6d has failed:

On branch mergify/bp/kilted/pr-71
Your branch is up to date with 'origin/kilted'.

You are currently cherry-picking commit 383fc6d.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   .github/workflows/ci.yml

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   .github/workflows/nightly.yml

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@mergify mergify Bot added the conflicts label Sep 11, 2026
@mergify mergify Bot mentioned this pull request Sep 11, 2026
@mergify

mergify Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor Author

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.06%. Comparing base (62763fb) to head (28de8c1).

Additional details and impacted files
@@             Coverage Diff             @@
##           kilted      #72       +/-   ##
===========================================
+ Coverage   59.43%   80.06%   +20.62%     
===========================================
  Files          50        2       -48     
  Lines        5350      311     -5039     
  Branches     2320       77     -2243     
===========================================
- Hits         3180      249     -2931     
+ Misses        670       48      -622     
+ Partials     1500       14     -1486     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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

@David-LP99
David-LP99 merged commit 7cb327d into kilted Sep 14, 2026
8 checks passed
@David-LP99
David-LP99 deleted the mergify/bp/kilted/pr-71 branch September 14, 2026 05:30
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