Skip to content

Commit 508b6d4

Browse files
NullVoxPopuliclaude
andcommitted
Fix ember 7 beta/canary compat: service inject, @glimmer/component
- Replace `inject as injectService` with `service as injectService` from @ember/service (inject was removed in ember 7) - Update @glimmer/component to ^2.0.0 (v1 imports from the ember barrel which was removed in ember 7) Verified locally: lints pass, 553/553 tests pass. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 9d1e747 commit 508b6d4

3 files changed

Lines changed: 20 additions & 134 deletions

File tree

test-app/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@
3838
"@ember/test-waiters": "^4.0.0",
3939
"@embroider/macros": "^1.20.2",
4040
"@embroider/test-setup": "^3.0.3",
41-
"@glimmer/component": "^1.1.2",
41+
"@glimmer/component": "^2.0.0",
4242
"@glimmer/tracking": "^1.1.2",
4343
"concurrently": "^8.0.1",
4444
"ember-auto-import": "^2.13.1",

test-app/pnpm-lock.yaml

Lines changed: 18 additions & 132 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

test-app/tests/unit/setup-context-test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { module, test } from 'qunit';
22
import { isTesting } from '@ember/debug';
3-
import Service, { inject as injectService } from '@ember/service';
3+
import Service, { service as injectService } from '@ember/service';
44
import {
55
setupContext,
66
teardownContext,

0 commit comments

Comments
 (0)