protocolbuffers / protocolbuffers/protobuf-javascript
Should `goog.DEBUG` be false for builds distributed via npm?
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 471
- Forks
- 91
- Merge medio
- 3 h 57 min
- PR fusionados (30 d)
- 2
Descripción
goog.DEBUG is true in the google-protobuf.js runtime distributed in the npm package. This setting gates the various assertion code in the runtime (goog.asserts.ENABLE_ASSERTS = goog.DEBUG;).
- Is this expected?
- If so, should DEBUG be set to false for production builds? And should two builds be distributed, or a means be exposed to configure this?
- If not, should the gulpfile that generates the runtime supply
--define goog.DEBUG=falseto the closure compiler invocation?
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Lee closure/goog/base.js en las líneas 224-230 e inspecciona la invocación de Closure Compiler en el gulpfile. Compara el runtime google-protobuf.js generado en el paquete npm con el comportamiento de producción previsto y determina si el build debería cambiarse, si debería exponerse la configuración o si deberían distribuirse varios builds.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript
- Área
- build-system
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 25/100