protocolbuffers / protocolbuffers/protobuf-javascript
Allow for custom npm packages in imported protos
Nessuno ha ancora preso questa issue.
- Lingua principale
- JavaScript
- Stelle
- 471
- Fork
- 91
- Merge medio
- 3h 57m
- PR unite (30g)
- 2
Descrizione
Here is a link to the line where it speaks to our issue.
https://github.com/protocolbuffers/protobuf-javascript/blob/6113588085fea6149c51a750bd21b5cf9334a07d/generator/js_generator.cc#L116-L119
We currently make use of a variety of google protos other than those in the google-protobuf npm package. To make use of these, we have to explicitly include them in the npm package that we publish containing our generated protobufs. It would be ideal if we could publish the generated google protobufs we use in a separate npm package and use options in the .proto files to specify the name of the package to which these are published, and for this package to be used in the generated code whenever one of these protobufs is implemeneted. Similar to how this is for the "google-protobuf" imports
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia da generator/js_generator.cc alle righe 116-119, collegato nell'issue, e traccia come vengono emessi gli import per google-protobuf. Determina come le opzioni .proto debbano specificare i nomi dei package per gli altri package protobuf generati; il lavoro è completato quando il codice generato usa i nomi dei package configurati per i relativi import.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- javascript
- Ambito
- devtools
- Tipo di issue
- Funzionalità
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 30/100