ember-cli / ember-cli/loader.js
Avoid mutating module exports.
- Lenguaje dominante
- JavaScript
- Estrellas
- 79
- Forks
- 69
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
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.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Línea de trabajo
Comienza en lib/loader/loader.js, en makeDefaultExport, y revisa el contexto de migración enlazado de ember-cli. Sigue cómo se mutan actualmente las exportaciones de módulos en las compilaciones de depuración y producción, y verifica después que el comportamiento de compatibilidad y la ruta de deprecación coincidan con el plan loader.js@5.0.0 de la issue.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript
- Área
- tooling
- Tipo de issue
- Refactorización
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100