ember-cli / ember-cli/loader.js

Avoid mutating module exports.

Abierto
#114 4 comentarios 3 reacciones 0 asignados Ver en GitHub
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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.