[FEATURE] Move htmlSafe and isHTMLSafe to @ember/component

[Serabe]

Related to emberjs/rfcs#236.

Task in emberjs/ember.js#15624

[Turbo87]

We can't do this. This is a breaking change!

[Serabe]

With current format, there cannot be two shims for the same function. Shall I change the format?

[Turbo87]

Changing the format is also a breaking change.

[locks]

Is this project not on v0.2.7?

[Turbo87]

v0.2.7 of what? I don't understand the question

[martletandco]

How does this impact helpers? htmlSafe and friends fall under templates rather than just components. You can import @ember/component into a helper but that shows it's a little off

[Serabe]

Hello @martletandco:

That's an interesting point. Can you repeat it in emberjs/rfcs#236 ? I think it is worth it :)

Thank you!

[rwjblue]

We can't do this. This is a breaking change!

Sure we can, we just need a major version bump here.

[rwjblue]

IMHO, the current JSON format needs a bit of reworking to accommodate multiple import paths pointing to a single global. We should focus on deciding what the new format is, so that the underlying work can begin.

[rwjblue]

My suggestion:

[
  { 
    "global": "Ember.String.isHTMLSafe", 
    "module": "@ember/component", 
    "export": "isHTMLSafe", 
    "deprecated": true,
    "replacement": { "module": "@ember/template", "export": "isHTMLSafe" }
  },
  { 
    "global": "Ember.String.isHTMLSafe", 
    "module": "@ember/template", 
    "export": "isHTMLSafe", 
    "deprecated": false
  },
  { 
    "global": "Ember.Component", 
    "module": "ember-component", 
    "export": "default", 
    "deprecated": true,
    "replacement": { "module": "@ember/component", "export": "default" }
  },
  { 
    "global": "Ember.Component", 
    "module": "@ember/component", 
    "export": "default", 
    "deprecated": false
  }
]

[locks]

I believe #38 supersedes this one.