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

Enable --fast CLI flag for transpile mode only #130

Merged
merged 1 commit into from
Jun 11, 2016
Merged

Conversation

blakeembrey
Copy link
Member

Related to #44.

Without --fast tests on typings-core:

npm run test-spec  7.98s user 0.52s system 72% cpu 11.691 total

With --fast tests on typings-core:

npm run test-spec  3.52s user 0.47s system 56% cpu 7.098 total

@coveralls
Copy link

coveralls commented Jun 11, 2016

Coverage Status

Coverage decreased (-27.6%) to 60.262% when pulling 4f7df7c on fast-option into a1384b6 on master.

@coveralls
Copy link

coveralls commented Jun 11, 2016

Coverage Status

Coverage decreased (-1.3%) to 86.638% when pulling 58d3e6b on fast-option into a1384b6 on master.

@blakeembrey blakeembrey merged commit efcd2fd into master Jun 11, 2016
@blakeembrey blakeembrey deleted the fast-option branch June 11, 2016 05:06
@mjohnsonengr
Copy link

1.5x is an understatement. Upgrading from 0.8 -> 0.9.1 got my test suite from 3.5 minutes to 22 seconds; the --fast flag then took me down to 2 seconds!

Awesome work and cheers @blakeembrey!! 😁

@blakeembrey
Copy link
Member Author

Wow, that is ridiculous! Sorry it took so long for that improvement to make it's way in 😄

@mjohnsonengr
Copy link

Hah! No worries! I just now started working on getting our test suites to run in node last week and only hit the performance issues this morning before I found this upgrade! (previously it was a mess of hacks making them run with compiled JS coverage in the browser)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants