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

[master] Workflow updates #22

Merged
merged 4 commits into from
May 13, 2020
Merged

[master] Workflow updates #22

merged 4 commits into from
May 13, 2020

Conversation

chapulina
Copy link
Contributor


⚠️ This is using this PR, change to master once that's merged: gazebo-tooling/action-gz-ci#7

Signed-off-by: Louise Poubel <[email protected]>
@chapulina chapulina requested a review from iche033 as a code owner May 8, 2020 23:05
@github-actions github-actions bot added the 🔮 dome Ignition Dome label May 8, 2020
@iche033
Copy link
Contributor

iche033 commented May 11, 2020

@chapulina what's the magic you did to make codecov comment on the PR like in gazebosim/gz-rendering#80?

Signed-off-by: Louise Poubel <[email protected]>
@chapulina
Copy link
Contributor Author

what's the magic

The main magic that was missing in general was installing the codecov app, which I did for all Ignition repos. This specific branch wasn't even running tests, so that was a different problem. Should be temporarily fixed by 0596a74

@codecov
Copy link

codecov bot commented May 11, 2020

Codecov Report

❗ No coverage uploaded for pull request base (master@95e1b08). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #22   +/-   ##
=========================================
  Coverage          ?   59.63%           
=========================================
  Files             ?       16           
  Lines             ?      976           
  Branches          ?        0           
=========================================
  Hits              ?      582           
  Misses            ?      394           
  Partials          ?        0           

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 95e1b08...136bce1. Read the comment docs.

chapulina added 2 commits May 11, 2020 18:35
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
@chapulina chapulina merged commit 7affa6a into master May 13, 2020
@chapulina chapulina deleted the workflow_updates_master branch May 13, 2020 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔮 dome Ignition Dome
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants