protocolbuffers / protocolbuffers/protobuf-javascript
Should `goog.DEBUG` be false for builds distributed via npm?
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 471
- Forks
- 91
- Ø Merge
- 3 Std. 57 Min.
- Gemergte PRs (30 T.)
- 2
Beschreibung
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?
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Lies closure/goog/base.js in den Zeilen 224–230 und prüfe den Aufruf des Closure Compiler im gulpfile. Vergleiche die generierte google-protobuf.js-Laufzeitumgebung im npm-Paket mit dem vorgesehenen Verhalten in der Produktion und bestimme anschließend, ob der Build geändert werden sollte, Konfiguration verfügbar gemacht werden sollte oder mehrere Builds verteilt werden sollten.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript
- Bereich
- build-system
- Issue-Typ
- Feature
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 25/100