protocolbuffers / protocolbuffers/protobuf-javascript
js/{closure,commonjs}: create a new js_module option
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
What language does this apply to?
If it's a proto syntax change, is it for proto2 or proto3? proto3
If it's about generated code change, what programming language? javascript/commonjs but it can be applied to es6 or closure as well.
Describe the problem you are trying to solve.
We are generating code of multiple packages that we want to publish independently but with references between them. Currently the generator creates a tightly coupled relative paths between folders.
Describe the solution you'd like
An option like Go's go_package or C#'s csharp_namespace where I can put the import path, in this case the JS module.
I think a js_module called option would do the trick.
Describe alternatives you've considered
Overriding the generated code.
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
Comienza leyendo cómo el generador de JavaScript gestiona las importaciones relativas para las salidas CommonJS, ES6 y Closure; después, revisa la discusión de este issue sobre la semántica propuesta de js_module. El trabajo estará terminado cuando una opción pueda establecer rutas de importación de módulos independientes y conservar las referencias entre los paquetes generados en todos los destinos de JavaScript compatibles.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript
- Área
- tooling
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 30/100