Skip to content

docs(ember): add deprecation guide for Comparable mixin - #1432

Merged
NullVoxPopuli merged 5 commits into
ember-learn:mainfrom
olenderhub:add-comparable-mixin-deprecation-guide
Sep 1, 2026
Merged

NullVoxPopuli merged 5 commits into
ember-learn:mainfrom
olenderhub:add-comparable-mixin-deprecation-guide

Conversation

@olenderhub

Copy link
Copy Markdown
Contributor

Adds the deprecation guide for deprecate-comparable-mixin.

This matches the deprecation entry added in Ember:

  • id: deprecate-comparable-mixin
  • since: 7.2.0
  • until: 8.0.0
  • URL: https://deprecations.emberjs.com/id/deprecate-comparable-mixin

The guide explains that apps and addons should stop importing/extending the Comparable mixin and instead define a function-valued compare(other) method directly.

Refs emberjs/ember.js#21437

@netlify

netlify Bot commented Jun 4, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for ember-deprecations ready!

Name Link
🔨 Latest commit dba6bcd
🔍 Latest deploy log https://app.netlify.com/projects/ember-deprecations/deploys/6a96f3d62118b4000817755f
😎 Deploy Preview https://deploy-preview-1432--ember-deprecations.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Add migration guide for the Comparable mixin deprecation in Ember v6, showing how to replace mixin usage with a direct compare() method.
@NullVoxPopuli
NullVoxPopuli force-pushed the add-comparable-mixin-deprecation-guide branch from 41573ee to e22c88f Compare August 31, 2026 23:26
@NullVoxPopuli
NullVoxPopuli force-pushed the add-comparable-mixin-deprecation-guide branch from e22c88f to 9880088 Compare August 31, 2026 23:27
---
title: Deprecate Comparable mixin
until: 8.0.0
since: 7.2.0

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@NullVoxPopuli did this actually land in 7.2?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It did, but it is also until 7.5 because it was private, so just an intimate api

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

updated

Comment thread content/ember/v7/deprecate-comparable-mixin.md Outdated
@NullVoxPopuli
NullVoxPopuli merged commit 40f1366 into ember-learn:main Sep 1, 2026
4 checks passed
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.

4 participants