ember-compatibility-helpers icon indicating copy to clipboard operation
ember-compatibility-helpers copied to clipboard

Only Runs On Addon Directory

Open chadhietala opened this issue 7 years ago • 5 comments

This seems to only run on the addon directory when used in an addon. When I moved the code that was using ember-compatibilty-helpers to the addon directory the inlining of the boolean worked. Prior to that I would get a runtime exception about ember-compatibilty-helpers missing.

chadhietala avatar Nov 26 '18 16:11 chadhietala

That makes some sense, this addon is a plugin that adds itself to its parent's babel configuration, so maybe it's only running against the /addon dir because that's all that ember-cli-babel does when included in an Ember addon? e.g., app gets merged directly into the root app without any transforms being run on it

pzuraq avatar Nov 26 '18 16:11 pzuraq

This seems to only run on the addon directory when used in an addon. When I moved the code that was using ember-compatibilty-helpers to the addon directory the inlining of the boolean worked.

Yep, sounds like things are functioning as intended. This is exactly the reason that we advise folks that all addon code should be in the addon/ directory, and the app/ directory should only contain re-exports.

rwjblue avatar Nov 26 '18 17:11 rwjblue

I experienced the opposite behavior (not running in the addon directory) and reported the issue at #32.

ppcano avatar Feb 07 '19 09:02 ppcano

This seems to only run on the addon directory when used in an addon. When I moved the code that was using ember-compatibilty-helpers to the addon directory the inlining of the boolean worked.

Yep, sounds like things are functioning as intended. This is exactly the reason that we advise folks that all addon code should be in the addon/ directory, and the app/ directory should only contain re-exports.

What about the fastboot/ directory? I think I am hitting this issue for code in fastboot/:

Could not find module `ember-compatibility-helpers` imported from `<PATH TO FILE THAT IMPORTS ember-compatibility-helpers>`

rileyhilliard avatar Jun 11 '20 17:06 rileyhilliard

fastboot is roughly the same treatment as app I think (though we'd have to double check with the implementation in ember-cli-fastboot).

rwjblue avatar Jul 15 '20 02:07 rwjblue