Skip to content

Migrate to eProsima CI - #69

Merged
jparisu merged 6 commits into
mainfrom
ci/migrate-eprosima-ci
Jun 13, 2023
Merged

Migrate to eProsima CI#69
jparisu merged 6 commits into
mainfrom
ci/migrate-eprosima-ci

Conversation

@jparisu

@jparisu jparisu commented Jun 13, 2023

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: jparisu <javierparis@eprosima.com>
@jparisu
jparisu temporarily deployed to codecov June 13, 2023 05:54 — with GitHub Actions Inactive
@codecov-commenter

codecov-commenter commented Jun 13, 2023

Copy link
Copy Markdown

Codecov Report

Patch coverage has no change and project coverage change: +1.97 🎉

Comparison is base (79ceb40) 72.31% compared to head (f0e309f) 74.29%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #69      +/-   ##
==========================================
+ Coverage   72.31%   74.29%   +1.97%     
==========================================
  Files          59       59              
  Lines        1156     1268     +112     
  Branches      287      292       +5     
==========================================
+ Hits          836      942     +106     
- Misses        158      161       +3     
- Partials      162      165       +3     

see 31 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Comment thread .github/workflows/test.yml Outdated
# - ubuntu-22.04
# - test python linter
#
# TODO: when merge CI in eprosima-CI repository, the jparisu fork must change for eProsima in actions.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove?

Comment thread .github/workflows/test.yml Outdated
#
# TODO: when merge CI in eprosima-CI repository, the jparisu fork must change for eProsima in actions.
#
# TODO: Add Windows CI

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove?

@jparisu
jparisu temporarily deployed to codecov June 13, 2023 06:33 — with GitHub Actions Inactive
Signed-off-by: jparisu <javierparis@eprosima.com>
@jparisu
jparisu force-pushed the ci/migrate-eprosima-ci branch from a2e46bc to d1fb274 Compare June 13, 2023 06:40
@jparisu
jparisu temporarily deployed to codecov June 13, 2023 06:40 — with GitHub Actions Inactive
Signed-off-by: jparisu <javierparis@eprosima.com>
@jparisu
jparisu temporarily deployed to codecov June 13, 2023 06:43 — with GitHub Actions Inactive
Signed-off-by: jparisu <javierparis@eprosima.com>
Signed-off-by: jparisu <javierparis@eprosima.com>
@jparisu
jparisu temporarily deployed to codecov June 13, 2023 07:01 — with GitHub Actions Inactive
@jparisu
jparisu temporarily deployed to codecov June 13, 2023 07:22 — with GitHub Actions Inactive
Signed-off-by: jparisu <javierparis@eprosima.com>
@jparisu
jparisu force-pushed the ci/migrate-eprosima-ci branch from f0e309f to f136bb3 Compare June 13, 2023 07:35
@jparisu
jparisu temporarily deployed to codecov June 13, 2023 07:35 — with GitHub Actions Inactive

@irenebm irenebm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jparisu
jparisu merged commit 31a6881 into main Jun 13, 2023
@jparisu
jparisu deleted the ci/migrate-eprosima-ci branch June 13, 2023 14:31
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.

3 participants