Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for using $ as a separator in addition to ::. #203

Merged
merged 2 commits into from
Aug 15, 2019

Conversation

rwjblue
Copy link
Owner

@rwjblue rwjblue commented Aug 15, 2019

Since Ember has decided to repurpose the :: sigil to mean "nested folder invocation" (see emberjs/rfcs#457), this introduces $ as a replacement sigil. Ultimately, usage of :: will be deprecated by this addon (though that will be in a future commit).

Implements part of #202

@rwjblue rwjblue force-pushed the add-sigil-support branch from f55e924 to 4fba465 Compare August 15, 2019 13:57
Since Ember has decided to repurpose the `::` sigil to mean "nested
folder invocation" (see emberjs/rfcs#457), this introduces `$` as a
replacement sigil. Ultimately, usage of `::` will be deprecated by this
addon (though that will be in a future commit).
@rwjblue rwjblue force-pushed the add-sigil-support branch from 4fba465 to 2b1cb7b Compare August 15, 2019 14:11
Skip tests for invoking angle bracket components due to
incompatibilities with ember-angle-bracket-invocation-polyfill 1.3 and
higher (since it added support for `::` syntax).
@rwjblue rwjblue merged commit a03698b into master Aug 15, 2019
@rwjblue rwjblue deleted the add-sigil-support branch August 15, 2019 14:54
@rwjblue rwjblue mentioned this pull request Aug 15, 2019
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant