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

(chore): Fix codecov #5554

Merged
merged 1 commit into from
May 4, 2019
Merged

(chore): Fix codecov #5554

merged 1 commit into from
May 4, 2019

Conversation

dplewis
Copy link
Member

@dplewis dplewis commented May 2, 2019

@codecov
Copy link

codecov bot commented May 2, 2019

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #5554   +/-   ##
=========================================
  Coverage          ?   94.06%           
=========================================
  Files             ?      129           
  Lines             ?     9210           
  Branches          ?        0           
=========================================
  Hits              ?     8663           
  Misses            ?      547           
  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 1b0d20d...22526a3. Read the comment docs.

@dplewis dplewis requested a review from acinader May 2, 2019 13:23
@dplewis dplewis changed the title (chore): Fix broken codecov (chore): Fix codecov May 2, 2019
@acinader
Copy link
Contributor

acinader commented May 2, 2019

any idea why it was broken and why this fixes it??

@dplewis
Copy link
Member Author

dplewis commented May 2, 2019

It might be an internal fix for exclude and include.

As you can see /src was excluded before.

Copy link
Contributor

@acinader acinader left a comment

Choose a reason for hiding this comment

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

i don't quite get it, but no harm!

@acinader acinader merged commit 87b0093 into master May 4, 2019
@dplewis
Copy link
Member Author

dplewis commented May 4, 2019

Story of my life lol

@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 7.1.0-alpha.4

@parseplatformorg parseplatformorg added the state:released-alpha Released as alpha version label Mar 31, 2024
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 7.1.0-beta.1

@parseplatformorg parseplatformorg added the state:released-beta Released as beta version label Jun 30, 2024
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 7.1.0

@parseplatformorg parseplatformorg added the state:released Released as stable version label Jun 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
state:released Released as stable version state:released-alpha Released as alpha version state:released-beta Released as beta version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants