Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

Remove dependency on fs-plus to reduce load time #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Remove dependency on fs-plus to reduce load time #4

wants to merge 1 commit into from

Conversation

chocolateboy
Copy link

This PR improves the responsiveness of coffee-cash by removing its dependency on fs-plus. It also expands the version range of the coffee-script dependency to include 1.9, 1.10 &c. (1.8.0 is over a year old).

In my particular use case, this yields a speedup of ~20 ms (15% of the total execution time, including node's slow startup, and > 50% of the application execution time). This is unlikely to be significant in Atom, but it can make a huge difference on the command line.

$ cat ./profile.js
require('time-require')
require('coffee-cash')
$ node ./profile.js

time-require

open up coffee-script dependency version to include 1.9, 1.10 &c.
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.

1 participant