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

test(exoflex): make tests even FASTER, BLAZING FAST ⚡️ ⚡️ ⚡️ #176

Merged
merged 2 commits into from Nov 2, 2019
Merged

test(exoflex): make tests even FASTER, BLAZING FAST ⚡️ ⚡️ ⚡️ #176

merged 2 commits into from Nov 2, 2019

Conversation

oshimayoan
Copy link
Contributor

@oshimayoan oshimayoan commented Nov 1, 2019

⚡️ ⚡️ ⚡️

In this diff, I try to make our tests become even faster:

jest time
ts-jest --no-cache 160s
ts-jest w/patch --no-cache 89s
ts-jest w/patch 35s
babel --no-cache 140s
babel w/patch --no-cache 70s
babel w/patch 19s

NOTE: Have to rebase this after #174 landed.

@oshimayoan oshimayoan added this to In progress in exoflex via automation Nov 1, 2019
Copy link
Member

@darcien darcien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you just say FAST 👀? I'm sold.

🐑 it! 🔥 🔥 🔥

When we get a stable release that doesn't need a patch, maybe we should also update the config in our repo templates.

@@ -59,6 +61,8 @@
"jest-watch-select-projects": "^1.0.0",
"jest-watch-typeahead": "^0.4.0",
"jest-with-platform": "^0.0.3",
"patch-package": "^6.2.0",
"postinstall-postinstall": "^2.0.0",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this a real package name? 😆

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah hahaha

@oshimayoan oshimayoan changed the title test(exoflex): make tests even FASTER, BLAZING FAST ⚡️ ⚡️ ⚡️ test(exoflex): make tests even FASTER, BLAZING FAST ⚡️ ⚡️ ⚡️ Nov 2, 2019
@oshimayoan oshimayoan merged commit 70cf1ae into kodefox:master Nov 2, 2019
exoflex automation moved this from In progress to Done Nov 2, 2019
@oshimayoan oshimayoan deleted the babel-preset-typescript branch November 2, 2019 03:06
oshimayoan added a commit that referenced this pull request Nov 6, 2019
* test(exoflex): remove ts-jest; add @babel/typescript, patch-package

* test(exoflex): patch @jest/transformer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
exoflex
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants