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

@ember-data/model: Simplify @cached transpilation #7599

Merged
merged 3 commits into from
Jul 22, 2021
Merged

@ember-data/model: Simplify @cached transpilation #7599

merged 3 commits into from
Jul 22, 2021

Conversation

dcyriller
Copy link
Contributor

@dcyriller dcyriller commented Jun 20, 2021

@dcyriller dcyriller marked this pull request as ready for review July 21, 2021 21:24
@igorT igorT added 🎯 canary PR is targeting canary (default) 🏷️ chore This PR primarily refactors code or updates dependencies labels Jul 22, 2021
@igorT
Copy link
Member

igorT commented Jul 22, 2021

Perf tests fail due to #7639, should pass after that

@igorT
Copy link
Member

igorT commented Jul 22, 2021

Beta fails on master right now.

@igorT igorT merged commit 406d6b5 into emberjs:master Jul 22, 2021
@dcyriller dcyriller deleted the simplify-at-cached-transpilation branch July 22, 2021 18:27
igorT pushed a commit that referenced this pull request Jul 28, 2021
* Use patched ember-cached-decorator-polyfill

* Simplify `@cached` transpilation

* Use meber-cached-decorator-polyfill 0.1.4
igorT added a commit that referenced this pull request Jul 30, 2021
* Use patched ember-cached-decorator-polyfill

* Simplify `@cached` transpilation

* Use meber-cached-decorator-polyfill 0.1.4

Co-authored-by: Cyrille David <[email protected]>
igorT added a commit that referenced this pull request Aug 16, 2021
igorT added a commit that referenced this pull request Aug 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎯 canary PR is targeting canary (default) 🏷️ chore This PR primarily refactors code or updates dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants