codecov[bot]
commented
5 years ago Codecov Report
Merging #41 into master will decrease coverage by
0.05%. The diff coverage is80.95%.
@@ Coverage Diff @@
## master #41 +/- ##
==========================================
- Coverage 83.07% 83.01% -0.06%
==========================================
Files 63 64 +1
Lines 1861 1878 +17
==========================================
+ Hits 1546 1559 +13
- Misses 199 202 +3
- Partials 116 117 +1| Impacted Files | Coverage Δ | |
|---|---|---|
| pkg/core/command/result.go | 60% <ø> (ø) |
:arrow_up: |
| pkg/dicebot/testing/testing.go | 65.71% <100%> (+4.42%) |
:arrow_up: |
| pkg/bcdice/bcdice.go | 80.85% <71.42%> (-2.87%) |
:arrow_down: |
| pkg/core/util/secret_roll.go | 77.77% <77.77%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update f45435b...ba6b8e3. Read the comment docs.
（上方無限ロールの続きですので、#38 のマージ後の確認をお願いします）
シークレットロール（
S）を実装しました。Ruby版と同様に、Sまたはsが先頭についていたとき、その部分を除いて構文解析を行います（そのため、これまでの構文解析処理は変更していません）。結果の構造体に追加したIsSecretというフラグを呼び出し側が管理する形です。そのままだと特にREPLで変化が分かりにくいため、REPLではシークレットロールかどうかの表示を行うようにしました。