Skip to content

Do not error out if coverage upload fails#6380

Merged
DonJayamanne merged 2 commits intomicrosoft:masterfrom
kimadeline:6340-coverage-upload
Jul 1, 2019
Merged

Do not error out if coverage upload fails#6380
DonJayamanne merged 2 commits intomicrosoft:masterfrom
kimadeline:6340-coverage-upload

Conversation

@kimadeline
Copy link
Copy Markdown

For #6340

  • Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
  • Title summarizes what is changing
  • [n/a] Has a news entry file (remember to thank yourself!)
  • [n/a] Appropriate comments and documentation strings in the code
  • [n/a] Has sufficient logging.
  • [n/a] Has telemetry for enhancements.
  • [n/a] Unit tests & system/integration tests are added/updated
  • [n/a] Test plan is updated as appropriate
  • [n/a] package-lock.json has been regenerated by running npm install (if dependencies have changed)
  • [n/a] The wiki is updated with any design decisions/details.

@kimadeline kimadeline added the no-changelog No news entry required label Jun 28, 2019
@DonJayamanne DonJayamanne merged commit 0920bce into microsoft:master Jul 1, 2019
@DonJayamanne
Copy link
Copy Markdown

Merging on behalf of Kim as this is a P0

@kimadeline kimadeline deleted the 6340-coverage-upload branch July 2, 2019 15:23
@lock lock Bot locked as resolved and limited conversation to collaborators Jul 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-changelog No news entry required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants