petehunt / node-jsx

transparently require() jsx from within node
164 stars 32 forks source link

Change default file extension to .jsx #34

Closed Josh-a-e closed 6 years ago

Josh-a-e commented 9 years ago

Performance improvements as now a much smaller number of files are transformed.

Appreciate this will have a potentially breaking impact on some implementations.

nemtsov commented 9 years ago

@Josh-a-e if you're interested in approx. the same performance improvement without breaking backwards compatibility, take a look at PR https://github.com/petehunt/node-jsx/pull/32.

sahat commented 9 years ago

While I am against breaking changes, switching extension to .jsx results in a significant performance increase. It took about 4-5s to start the server with .js extension on a quad-core Intel Core i7 2.5GHz. And now it's around 1.5s.