Skip to content

Deprecation enabled / URL updates for 7.5.0 - #21745

Merged
ef4 merged 6 commits into
mainfrom
nvp/enable-deprecations
Oct 26, 2026
Merged

ef4 merged 6 commits into
mainfrom
nvp/enable-deprecations

Conversation

@NullVoxPopuli

@NullVoxPopuli NullVoxPopuli commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

enable all implemented deprecations

@github-actions

github-actions Bot commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

📊 Size report

Tarball size — 1.3 MB → 1.3 MB

dist/dev   0.02%↑

File Before (Size / Brotli) After (Size / Brotli)
./packages/@ember/-internals/deprecations/index.js 10.6 kB / 2 kB 5%↑11.1 kB / -0.15%↓2 kB
./packages/shared-chunks/cache-{hash}.js 25 kB / 6.6 kB -97.2%↓697 B / -96.1%↓258 B
Total (Includes all files) 2.2 MB / 520.7 kB 0.02%↑2.2 MB / -0%↓520.7 kB

dist/prod   0.02%↑

File Before (Size / Brotli) After (Size / Brotli)
./packages/@ember/-internals/deprecations/index.js 10.3 kB / 1.9 kB 5%↑10.8 kB / -0.16%↓1.9 kB
Total (Includes all files) 2 MB / 477.5 kB 0.02%↑2 MB / -0%↓477.5 kB

smoke-tests/v2-app-template/dist   0.09%↑

File Before (Size / Brotli) After (Size / Brotli)
./assets/main-{hash}.js 324.1 kB / 86.1 kB 0.1%↑324.4 kB / 0.09%↑86.2 kB
Total (Includes all files) 359.6 kB / 97.1 kB 0.09%↑359.9 kB / 0.05%↑97.2 kB

smoke-tests/v2-app-hello-world-template/dist   0.2%↑

File Before (Size / Brotli) After (Size / Brotli)
./assets/main-{hash}.js 128.8 kB / 35.2 kB 0.2%↑129.2 kB / 0.07%↑35.3 kB
Total (Includes all files) 129.2 kB / 35.4 kB 0.2%↑129.5 kB / 0.09%↑35.4 kB

🤖 This report was automatically generated by wyvox/pkg-size

@NullVoxPopuli
NullVoxPopuli marked this pull request as ready for review October 20, 2026 00:42
@NullVoxPopuli

Copy link
Copy Markdown
Contributor Author

The only ones implemented, but aren't ready to bump their enabled are these ones: ember-learn/deprecation-app#1404

(PR needs review, and perhaps changes)

id: 'deprecate-evented',
for: 'ember-source',
since: { available: '7.3.0' },
since: { available: '7.3.0', enabled: '7.5.0' },

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The target action support one is fine but these others need the RFCs to go to 'Ready for release' aka Advance to released stage

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

you mean we can't release a deprecation that is available and enabled in the same version?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

or we just can't merge this PR until we advance all the corresponding RFC PRs on monday?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

the later ... the RFCs should be in the 'ready for release' stage, aka waiting to release

Copy link
Copy Markdown
Contributor Author

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Did we skip the FCP to go to Ready for Release?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We did, yea -- it didn't occur to anyone on the call that it would be needed

I do see the "Ready for Release" section says that FCP is required to move in to "Ready for Release" :(

https://github.com/emberjs/rfcs#ready-for-release

image

:(

@ef4
ef4 merged commit 0f0460c into main Oct 26, 2026
70 checks passed
@ef4
ef4 deleted the nvp/enable-deprecations branch October 26, 2026 16:27
NullVoxPopuli added a commit to emberjs/ember-inspector that referenced this pull request Nov 2, 2026
ember.js 7.5 enabled the deprecation of `A` from `@ember/array`
(emberjs/ember.js#21745). In the ember-canary scenario, the
"Updates are published when they happen" test called `emberA()`.
ember-debug then sent `deprecation:count` after `promisesUpdated`,
so the test read `message.promises` from the wrong message and
threw.

The `emberA()` wrappers only fed native `find`, and no test visits
the route whose model used `A()`.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.

3 participants