Skip to content

WIP - Kill the test process on CI, 10s after completion#4909

Merged
DonJayamanne merged 24 commits into
microsoft:masterfrom
DonJayamanne:killmocha
Apr 2, 2019
Merged

WIP - Kill the test process on CI, 10s after completion#4909
DonJayamanne merged 24 commits into
microsoft:masterfrom
DonJayamanne:killmocha

Conversation

@DonJayamanne
Copy link
Copy Markdown

For #4905

  • Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
  • Title summarizes what is changing
  • Has a news entry file (remember to thank yourself!)
  • [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
  • 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.

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 25, 2019

Codecov Report

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

@@           Coverage Diff            @@
##             master   #4909   +/-   ##
========================================
  Coverage          ?     63%           
========================================
  Files             ?     372           
  Lines             ?   14624           
  Branches          ?    1159           
========================================
  Hits              ?    9068           
  Misses            ?    5348           
  Partials          ?     208

@DonJayamanne DonJayamanne changed the title Kill the test process on CI, 10s after completion WIP - Kill the test process on CI, 10s after completion Apr 1, 2019
@DonJayamanne DonJayamanne merged commit 71444b5 into microsoft:master Apr 2, 2019
@DonJayamanne DonJayamanne deleted the killmocha branch May 24, 2019 20:21
@lock lock Bot locked as resolved and limited conversation to collaborators Jul 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants