Skip to content

Fix CI - #713

Merged
ef4 merged 1 commit into
mainfrom
nvp/fix-ci
May 6, 2026
Merged

Fix CI#713
ef4 merged 1 commit into
mainfrom
nvp/fix-ci

Conversation

@NullVoxPopuli

@NullVoxPopuli NullVoxPopuli commented May 6, 2026

Copy link
Copy Markdown
Contributor

note:

  • fastboot is not compatible with ember 7
  • I don't know how to support fixing CI, and having fastboot tests work at all, unless we just don't test fastboot against beta and canary
    • in particular, when I patch ember-cli-fastboot, I then get errors from ancient ember (3.4) not allowing import * as name......
      so, I'm disabling fastboot on beta and canary

NullVoxPopuli added a commit that referenced this pull request May 6, 2026
Ember 6.10 introduced the AMD deprecation (RFC #1101): the published
ember-source package will stop bundling pre-built AMD modules after
Ember 7. On canary and beta, addons that still `import Ember from
'ember'` fail at boot with `Could not find module 'ember'`.

Two addons in the app/addon templates trip this:

- ember-resolver ^8.0.3: the v8 distribution exposes
  `ember-resolver/resolvers/classic/index` which imports the AMD
  `ember` module. Linking the canary scenario at v13.2.0 was already
  done in #713; do the same for beta.

- ember-cli-app-version ^5.0.0: `addon/initializer-factory.js` does
  `import Ember from 'ember'` to read `Ember.libraries`. v7.0.0
  switched to `import { libraries } from '@ember/-internals/metal'`.
  Link both canary and beta to ^7.0.0.

Both upgrades are scoped to the canary/beta scenarios via
`linkDevDependency` so the LTS/3.x/release scenarios continue to
exercise the older versions.

Verified locally: canary-babel, canary-csp, canary-doubly-indirect,
canary-layering, beta-babel, beta-common-chunk, beta-custom-html now
pass. Fastboot-using scenarios (canary-dynamic-import,
beta-import-sync, *-v2-addon) still fail because ember-cli-fastboot
4.1.5 (latest) imports `Ember` from AMD and uses the removed `inject
as service` shape — those need a separate fix.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@NullVoxPopuli
NullVoxPopuli marked this pull request as ready for review May 6, 2026 04:17
@ef4
ef4 merged commit f5165d2 into main May 6, 2026
171 checks passed
@ef4
ef4 deleted the nvp/fix-ci branch May 6, 2026 20:06
@github-actions github-actions Bot mentioned this pull request May 6, 2026
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.

2 participants