ember-cli / ember-cli/loader.js
Avoid mutating module exports.
- 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