ember-browserify icon indicating copy to clipboard operation
ember-browserify copied to clipboard

Build warnings from broccoli .read/.rebuild API

Open gabrielgrant opened this issue 6 years ago • 3 comments

After installing the default latest ember-browserify (v1.2.2) on a fresh app with ember-cli 3.1.4, building yields the following warnings:

[API] Warning: The .read and .rebuild APIs will stop working in the next Broccoli version
[API] Warning: Use broccoli-plugin instead: https://github.com/broccolijs/broccoli-plugin
[API] Warning: Plugin uses .read/.rebuild API: ember-browserify
[API] Warning: Plugin uses .read/.rebuild API: ember-browserify

gabrielgrant avatar May 27 '18 13:05 gabrielgrant

Yeah, it looks like there is some housekeeping to do to get up to date.

Does ember-cli-cjs-transform meet your needs? I suspect that most people who would be adding ember-browserify today are better served by that.

ef4 avatar May 29 '18 15:05 ef4

Ah, interesting, didn't realize browserify was no longer the recommended solution. It looks like ember-cli-cjs-transform should do the trick (though it is still pretty nice how this package allows quickly importing files without any config needed, but having to know how a package was installed in order to use obviously isn't ideal)

gabrielgrant avatar May 29 '18 19:05 gabrielgrant

Well I haven't "officially" made any recommendations to not use this yet, but I'm opening the discussion.

I agree that making things Just Work directly from module imports is the goal. The cjs transform is a necessary but not sufficient ingredient. The final step is invoking the cjs transform automatically whenever somebody does import from a module that appears in devDependencies or dependencies.

ef4 avatar May 30 '18 03:05 ef4