Skip to content
This repository was archived by the owner on May 13, 2019. It is now read-only.

coverage: remove folder #57

Merged
merged 1 commit into from
Jan 4, 2018
Merged

coverage: remove folder #57

merged 1 commit into from
Jan 4, 2018

Conversation

maclover7
Copy link
Contributor

Files have moved to nodejs/build repository.

Refs: nodejs/build#1055
Fixes: #56

Files have moved to nodejs/build repository.

Refs: nodejs/build#1055
Fixes: #56
@maclover7 maclover7 changed the title coverage: remote folder coverage: remove folder Jan 4, 2018
@maclover7
Copy link
Contributor Author

As an aside, I don't have push privileges on this repo, so someone else will have to land this for me :)

@Trott Trott merged commit 5d85de3 into nodejs:master Jan 4, 2018
@maclover7 maclover7 deleted the jm-rm-coverage branch January 4, 2018 04:26
@mhdawson
Copy link
Member

mhdawson commented Jan 4, 2018

@maclover7 I think this just broke coverage for the nightly jobs and running from the command line.

Can you revert until we get the rest of the pieces fixed up.

The content of some of the files is checked out so we need to update references to it.

https://ci.nodejs.org/job/node-test-commit-linux-coverage/488/nodes=benchmark/console

@mhdawson
Copy link
Member

mhdawson commented Jan 4, 2018

I'll put together a PR for master, but given that it may take a bit of time to propagate through the release we probably need to revert this for now.

mhdawson pushed a commit that referenced this pull request Jan 4, 2018
This reverts commit 5d85de3.

PR-URL: #58
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
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