ember-cli-typescript icon indicating copy to clipboard operation
ember-cli-typescript copied to clipboard

Need option to broaden exclusions from TS watcher

Open lupestro opened this issue 7 years ago • 5 comments

The CONCAT_STATS=true flag of broccoli-concat is used to collect the measured sizes of various pieces a built project and is instrumental in measuring svelting. It puts files in a concat-stats-for subdirectory, and the names of the files include directories with names like 2-vendor.js and 3-vendor.css.

With typescript in play, this hangs the build - perhaps it is being misled by directories that are named like JS and CSS files. Hand-editing the concat-stats-for directory into the list of exclusions in lib/utilities/compile.js enables the build to complete without an issue.

This is probably not the only situation where it would be desirable to exclude a directory from being watched by the TS build. An option to extend the list of exclusions would permit users encountering build issues to keep directories like this out of harm's way.

lupestro avatar May 29 '18 20:05 lupestro

@lupestro can you see whether this is still an issue in 1.5, and also whether it's an issue in the v2 beta?

chriskrycho avatar Nov 20 '18 02:11 chriskrycho

I’ll look into it tomorrow. It seems I’ve been able to run concat stats recently on a fresh machine without issues and without local hacks, but I need to see which version of ec-typescript our project is using. We’re getting into end game for this release, sitting at Ember 2.18 and possibly older TS as well, since we don’t want to “rock the boat”, but I can cut a branch to experiment in.

Full disclosure... Our project structure has also changed as we pulled our in-repo addons to separate addons, so we may no longer be testing the same project conditions.

Full details of what I find sometime tomorrow.

lupestro avatar Nov 20 '18 03:11 lupestro

Thanks!

chriskrycho avatar Nov 20 '18 03:11 chriskrycho

I didn’t get as far today as I’d hoped. The project and our addons for it are using ec-typescript 1.3.1 against typescript 2.9.1 - pretty far back. Things are getting busy at work before the holiday - it may be after Thanksgiving before I have experimental results with 1.5 for you.

lupestro avatar Nov 21 '18 03:11 lupestro

Even with the ec-typescript 1.3.1, with the in-repo addons gone, this builds fine with concat_stats turned on (just tried it), so results with 1.5 won’t be definitive. More news later, if you’re still interested.

lupestro avatar Nov 21 '18 14:11 lupestro

Going to close this as (1) it's long out of date and (2) we now recommend that people switch to using ember-cli-babel for apps (as documented here) and the rollup plugin configured as part of the v2 add-on build for add-ons, in conjunction with running glint or tac directly on their projects.

chriskrycho avatar Sep 28 '23 21:09 chriskrycho