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

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 <[email protected]>
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