Skip to content

Use codecov for checking coverage step instead of coveralls#1042

Merged
1 commit merged intogooddata:masterfrom
helobinvn:github-action
Jul 12, 2021
Merged

Use codecov for checking coverage step instead of coveralls#1042
1 commit merged intogooddata:masterfrom
helobinvn:github-action

Conversation

@helobinvn
Copy link
Copy Markdown
Contributor

No description provided.

@ghost
Copy link
Copy Markdown

ghost commented Jul 9, 2021

Build succeeded (check pipeline).

@helobinvn helobinvn added the do not merge Do not merge this yet label Jul 9, 2021
@ghost
Copy link
Copy Markdown

ghost commented Jul 9, 2021

Build succeeded (check pipeline).

@ghost
Copy link
Copy Markdown

ghost commented Jul 9, 2021

Build succeeded (check pipeline).

@ghost
Copy link
Copy Markdown

ghost commented Jul 9, 2021

Build succeeded (check pipeline).

1 similar comment
@ghost
Copy link
Copy Markdown

ghost commented Jul 12, 2021

Build succeeded (check pipeline).

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 12, 2021

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1042   +/-   ##
=========================================
  Coverage          ?   83.10%           
  Complexity        ?     2595           
=========================================
  Files             ?      343           
  Lines             ?     6634           
  Branches          ?      477           
=========================================
  Hits              ?     5513           
  Misses            ?      925           
  Partials          ?      196           

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 e831ae9...cf09fb7. Read the comment docs.

@ghost
Copy link
Copy Markdown

ghost commented Jul 12, 2021

Build succeeded (check pipeline).

@helobinvn helobinvn changed the title [DO NOT MERGE] verify commit SETI-5977: Use codecov for checking coverage step Jul 12, 2021
@helobinvn helobinvn removed the do not merge Do not merge this yet label Jul 12, 2021
@helobinvn helobinvn closed this Jul 12, 2021
@helobinvn helobinvn reopened this Jul 12, 2021
@ghost
Copy link
Copy Markdown

ghost commented Jul 12, 2021

Build succeeded (check pipeline).

Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml
@liry liry added the trivial label Jul 12, 2021
As coveralls.io is not supported yet for Github Action
so we switch to use codecov.io for our coverage check
@ghost
Copy link
Copy Markdown

ghost commented Jul 12, 2021

Build succeeded (check pipeline).

@helobinvn helobinvn changed the title SETI-5977: Use codecov for checking coverage step Use codecov for checking coverage step instead of coveralls Jul 12, 2021
@liry liry added the merge label Jul 12, 2021
@liry liry added this to the next milestone Jul 12, 2021
@ghost ghost removed the merge label Jul 12, 2021
@ghost ghost merged commit 59ca266 into gooddata:master Jul 12, 2021
@ghost
Copy link
Copy Markdown

ghost commented Jul 12, 2021

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants