protocolbuffers / protocolbuffers/protobuf-javascript
Should `goog.DEBUG` be false for builds distributed via npm?
オープン
まだ誰も着手していません。
help wanted
packaging & distribution
triaged
- 主要言語
- JavaScript
- スター
- 471
- フォーク
- 91
- 平均マージ
- 3時間 57分
- マージ済み PR（30日）
- 2
説明
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?
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
closure/goog/base.js の 224-230 行を読み、gulpfile の Closure Compiler の呼び出しを調べてください。npm パッケージ内で生成される google-protobuf.js のランタイムと、意図された本番環境での動作を比較し、そのうえで、ビルドを変更すべきか、設定を公開すべきか、または複数のビルドを配布すべきかを判断してください。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- javascript
- 領域
- build-system
- issue の種類
- 機能追加
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 活発さ
- 停滞
- 明瞭さ
- 説明が足りない
- 初心者へのやさしさ
- 25/100