Automattic / Automattic/VIP-Coding-Standards

CI: add code coverage check

オープン
#661 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
Type: Maintenance
主要言語
PHP
スター
261
フォーク
44
平均マージ
19分
マージ済み PR(30日)
1

説明

Follow up action after #628

Conversation about this between @GaryJones and @jrfnl from #628:

>>> Register with coveralls.io (or another service) and start recording and checking code coverage for builds.
>>
>> What sort of code coverage % do we currently have?
>
>![image](https://user-images.githubusercontent.com/663378/109984243-dcefa980-7d03-11eb-8320-3217e2ef7bd2.png)
>
>It's one of the things I've been monitoring locally when I have been working on individual sniffs.
>
>> Can this be done within the GitHub Actions rather than using an external service to display the results?
>
>Yes, we can use the `text` reporter instead of checkstyle to see the results in the action.
>
>We'd need to investigate how we can make the build fail on a decrease in code coverage in that case though. That part is what external services do automatically (and with configurable settings).

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

まず、リポジトリの GitHub Actions 設定と #628 の coverage に関する議論を確認します。coverage に Coveralls と action 内のテキスト reporter のどちらを使うべきか、現在の割合をどのように確立するか、そして低下時にどのように build を失敗させるべきかを判断します。build が coverage を記録し、合意した低下チェックを強制すれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
github-actions, php
領域
ci-cd, testing
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
30/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。