Skip to content

Remove Bower support.#492

Merged
rwjblue merged 1 commit intomasterfrom
remove-bower-components-support
Apr 9, 2020
Merged

Remove Bower support.#492
rwjblue merged 1 commit intomasterfrom
remove-bower-components-support

Conversation

@rwjblue
Copy link
Member

@rwjblue rwjblue commented Apr 9, 2020

Retrieving ember-template-compiler from bower_components is only required when using Ember versions older than 2.12. Bower itself is essentially deprecated, this removes the fallback code to use bower_components/ember/ember-template-compiler.js and improves the error message when we can't actually find ember-source.

You can still provide a custom path to the template compiler (and therefore use Bower if you wish) by:

// ember-cli-build.js

module.exports = function(defaults) {
  let app = new EmberApp(defaults, {
    'ember-cli-htmlbars': {
      templateCompilerPath: `some_path/to/ember-template-compiler.js`,
    }
  });
};

Closes #487

@rwjblue rwjblue force-pushed the remove-bower-components-support branch from 2263389 to 1b21e24 Compare April 9, 2020 13:03
Retrieving `ember-template-compiler` from `bower_components` is only
required when using Ember versions older than 2.12. Bower itself is
essentially deprecated, this removes the fallback code to use
`bower_components/ember/ember-template-compiler.js` and improves the
error message when we can't actually find `ember-source`.

You can still provide a custom path to the template compiler (and
therefore use Bower if you wish) by:

```js
// ember-cli-build.js

module.exports = function(defaults) {
  let app = new EmberApp(defaults, {
    'ember-cli-htmlbars': {
      templateCompilerPath: `some_path/to/ember-template-compiler.js`,
    }
  });
};
```
@rwjblue rwjblue force-pushed the remove-bower-components-support branch from 1b21e24 to 87c8886 Compare April 9, 2020 13:22
@rwjblue rwjblue merged commit 8766979 into master Apr 9, 2020
@rwjblue rwjblue deleted the remove-bower-components-support branch April 9, 2020 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Why does this still use bower?

1 participant