ember-cli / ember-cli/ember-load-initializers

Embroider crash when initializer/instance-initializer does not have a default export

Offen
#277 6 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
TypeScript
Sterne
16
Forks
33
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

If you have an initializer that does not have a default export:

```js
export function initialize() {}

// export default { initialize }
```

`resolveInitializer` crashes here:

https://github.com/ember-cli/ember-load-initializers/blob/b69de85ab66bfa0d857b8165e5bbc7769fba8382/addon/index.ts#L15-L16

This function only seems to be called when I enable Embroider builds, not in regular ember-cli builds, so I'm not sure if this is intentional or what the expectations are.

ember-cli could be stricter and require this default export, but if it's intentional to not require the default export, then `resolveInitializer` should handle this case as well?

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne in addon/index.ts bei resolveInitializer, insbesondere in den Zeilen 15–16, und reproduziere den Absturz mit dem Initializer-Beispiel aus dem Issue bei einem Embroider-Build. Prüfe, wie reguläre ember-cli-Builds ein Modul ohne Default-Export behandeln, und kläre, ob dieser Export erforderlich ist. Als erledigt gilt die Aufgabe, wenn die Erwartung im Verhalten dokumentiert ist und der gezeigte Fall nicht mehr abstürzt oder explizit abgelehnt wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript, typescript
Bereich
tooling
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.