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

Include polyfill for Ember 3.4 #126

Merged
merged 1 commit into from
May 18, 2022
Merged

Conversation

twokul
Copy link
Contributor

@twokul twokul commented May 17, 2022

This is a more conservative version of the existing PR: #123.

In Ember 3.4 (w/o the polyfill), angle-bracket invocations do not work properly in some cases. For example:

  • <FooBar class="some-class" />
  • <FooBar data-test="some-data-test" />

@twokul twokul marked this pull request as ready for review May 18, 2022 13:12
@twokul twokul force-pushed the twokul/splattrs branch 3 times, most recently from 8cc6f98 to 3d1be40 Compare May 18, 2022 15:42
@mmun
Copy link
Member

mmun commented May 18, 2022

Travis is running out of credits so we've switched to GitHub Actions for PRs too.

image

https://app.travis-ci.com/github/ember-polyfills/ember-angle-bracket-invocation-polyfill/requests

@twokul twokul force-pushed the twokul/splattrs branch 4 times, most recently from e2e8ee6 to f0f9eab Compare May 18, 2022 16:49
This is a more conservative version of the existing PR:
ember-polyfills#123.

In Ember 3.4 (w/o the polyfill), angle-bracket invocations do not work
properly in some cases. For example:

- `<FooBar class="some-class" />`
- `<FooBar data-test="some-data-test" />`
@mmun
Copy link
Member

mmun commented May 18, 2022

Fortunately, Ember 3.4 is passing. Ember 3.8 and Ember 3.28 are failing on two link-to related tests.

@mmun mmun merged commit 8e9ca2a into ember-polyfills:master May 18, 2022
@twokul twokul deleted the twokul/splattrs branch May 18, 2022 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants