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

Don't attempt to report if originating from a fork #43

Merged
merged 1 commit into from
Jul 13, 2020

Conversation

dabonnie
Copy link
Contributor

Signed-off-by: Devin Bonnie dbbonnie@amazon.com

Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>
@codecov
Copy link

codecov bot commented Jul 13, 2020

Codecov Report

Merging #43 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #43   +/-   ##
=======================================
  Coverage   25.43%   25.43%           
=======================================
  Files          26       26           
  Lines         798      798           
  Branches      185      185           
=======================================
  Hits          203      203           
  Misses        432      432           
  Partials      163      163           
Flag Coverage Δ
#unittests 25.43% <ø> (ø)

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 c9cbbad...87c7203. Read the comment docs.

@dabonnie dabonnie merged commit fbe81eb into master Jul 13, 2020
@dabonnie dabonnie deleted the dabonnie/cw-fork-action-fix branch July 13, 2020 17:32
@mm318
Copy link
Member

mm318 commented Jul 13, 2020

We keep merging this kind of change into our repos, but have we tested that it still reports build failures at all (issue brought up here and here)??

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.

None yet

5 participants