Skip to content

Add build matrix with CentOS to GH Actions#558

Merged
flongford merged 9 commits into
masterfrom
556-centos-gh-actions
Aug 3, 2021
Merged

Add build matrix with CentOS to GH Actions#558
flongford merged 9 commits into
masterfrom
556-centos-gh-actions

Conversation

@robertopreste

@robertopreste robertopreste commented Jul 30, 2021

Copy link
Copy Markdown
Contributor

Closes #557

This PR adds a second workflow that targets CentOS 7, only installing Symphony Remote on this platform. Tests are run separately in the Ubuntu workflow.

@codecov-commenter

codecov-commenter commented Aug 3, 2021

Copy link
Copy Markdown

Codecov Report

Merging #558 (ca2f30b) into master (4b07ecd) will not change coverage.
The diff coverage is n/a.

❗ Current head ca2f30b differs from pull request most recent head 6169a52. Consider uploading reports for the commit 6169a52 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master     #558   +/-   ##
=======================================
  Coverage   95.10%   95.10%           
=======================================
  Files          92       92           
  Lines        4168     4168           
  Branches      269      269           
=======================================
  Hits         3964     3964           
  Misses        144      144           
  Partials       60       60           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4b07ecd...6169a52. Read the comment docs.

@robertopreste robertopreste requested a review from flongford August 3, 2021 08:04

@flongford flongford 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.

Thanks @robertopreste! I only have a few minor comments, can re-review whenever

Comment thread .github/workflows/build_test_ubuntu18.yml Outdated
Comment thread .github/workflows/build_centos7.yml
Comment thread .github/workflows/build_centos7.yml
@robertopreste robertopreste requested a review from flongford August 3, 2021 10:08

@flongford flongford 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!

@flongford flongford merged commit ad9c61e into master Aug 3, 2021
@flongford flongford deleted the 556-centos-gh-actions branch August 3, 2021 11:04
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