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

fix: add ember-data version check to check if calling .toArray on ember-data arrays is deprecated #2525

Merged
merged 2 commits into from
Oct 27, 2023

Conversation

StephanH90
Copy link
Contributor

Calling .toArray on ember-data arrays is deprecated since [email protected]. Therefore we make the .toArray call conditional.

@StephanH90 StephanH90 self-assigned this Oct 4, 2023
@StephanH90 StephanH90 force-pushed the fix/add-ember-data-version-check branch 2 times, most recently from 05c18b9 to 67baab6 Compare October 26, 2023 09:45
@StephanH90 StephanH90 requested a review from anehx October 27, 2023 06:13
packages/core/package.json Outdated Show resolved Hide resolved
@StephanH90 StephanH90 force-pushed the fix/add-ember-data-version-check branch 3 times, most recently from c992ed2 to 4c90e8f Compare October 27, 2023 08:09
Copy link
Contributor

@anehx anehx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes in the lockfile seem very weird and don't make any sense to me. Could you reset the lockfile to main and run pnpm i again?

@StephanH90 StephanH90 force-pushed the fix/add-ember-data-version-check branch 2 times, most recently from 24d83e2 to 1deb79c Compare October 27, 2023 08:34
@StephanH90 StephanH90 force-pushed the fix/add-ember-data-version-check branch from 1deb79c to 75f3762 Compare October 27, 2023 08:44
@StephanH90
Copy link
Contributor Author

I had to downgrade my pnpm to 8.6.0 (I was on 8.8.0 before). Now the lockfile looks right.

@StephanH90 StephanH90 merged commit a719961 into main Oct 27, 2023
53 checks passed
@StephanH90 StephanH90 deleted the fix/add-ember-data-version-check branch October 27, 2023 08:56
anehx pushed a commit that referenced this pull request Nov 6, 2023
# [12.6.0](v12.5.0...v12.6.0) (2023-11-06)

### Bug Fixes

* add ember-data version check to check if calling `.toArray` on ember-data arrays is deprecated ([#2525](#2525)) ([a719961](a719961))

### Features

* **distribution:** pass distribution to abilities functions ([#2538](#2538)) ([1cf1593](1cf1593))
* enable single- and multiple-choice questions in the inquiry preview ([8c97c44](8c97c44))
@anehx
Copy link
Contributor

anehx commented Nov 6, 2023

🎉 This PR is included in version 12.6.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@anehx anehx added the released label Nov 6, 2023
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.

3 participants