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

build(package): remove copy test script #4161

Merged
merged 1 commit into from
Sep 25, 2018
Merged

Conversation

kwonoj
Copy link
Member

@kwonoj kwonoj commented Sep 20, 2018

Description:

I could see this script came in around ef35257#diff-b9cfc7f2cdf78a7f4b91a753d10865a2, but not sure around exact context - it seems it's not being used. This script copies over test cases under spec-build but no scripts are actually build / run using it, maybe we can remove and reduce some of build times?

Related issue (if exists):

@coveralls
Copy link

coveralls commented Sep 20, 2018

Coverage Status

Coverage decreased (-0.2%) to 96.809% when pulling 6fb244c on kwonoj:remove-copy into 02780dd on ReactiveX:master.

@benlesh benlesh merged commit 687a3ce into ReactiveX:master Sep 25, 2018
@kwonoj kwonoj deleted the remove-copy branch September 25, 2018 01:58
@lock lock bot locked as resolved and limited conversation to collaborators Oct 25, 2018
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