ember-cli / ember-cli/loader.js

Avoid mutating module exports.

Aperta
#114 4 commenti 3 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
JavaScript
Stelle
79
Fork
69
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

We should stop mutating the modules exports.

This was needed while ember-cli was using `amdStrict` mode (since it did not use the `_interopRequireDefault` babel helper), and that is being unrolled in https://github.com/ember-cli/ember-cli/pull/6827.

Suggested migration path is:

* Change [makeDefaultExport](https://github.com/ember-cli/loader.js/blob/master/lib/loader/loader.js#L127) to use `Object.defineProperty(exports, 'default,...)` with a getter that issues a deprecation warning in debug builds (avoiding `Object.defineProperty` in prod builds).
* Remove deprecation and method after major version bump to loader.js@5.0.0.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Inizia in lib/loader/loader.js, da makeDefaultExport, e rivedi il contesto di migrazione collegato di ember-cli. Traccia come vengono attualmente mutati gli export dei moduli nelle build di debug e di produzione, quindi verifica che il comportamento di compatibilità e il percorso di deprecazione corrispondano al piano loader.js@5.0.0 dell'issue.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
javascript
Ambito
tooling
Tipo di issue
Refactoring
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.