レビュアーがなにをどうレビューすればよいかがわからない

[takemikami]

もっと気軽にレビューに参加して欲しいが、
レビューにかける工数もさることながら，なにをレビューすればいいのか？が分からないがゆえに、レビューに参加しにくくなっていると思われる

• ソースがあるかどうか？
• ソースが複数あるかどうか？
• ソースがそもそも正しいかどうか？ (解釈違いがあり議論を呼ぶ内容なのかどうか？)
• ソース通りXMLに入力できたか？ (単数の値のカンマ繋ぎとか，どうやったのか？とか)
• XMLに入力する方法に再現性はあるのか？たとえばexcel経由で生成しているならそのソースコード？

対象がデータなので，テストが書かれているか？みたいなのは直接的には無いとしても，こういうクエリでこういう値が取れます，みたいなのが提示されると，データの正しさを確認するのは容易にできるかもしれない
PR出てるブランチに対して，PR上からクエリ投げられるようになったりするとよいかもしれない。
（現状は、ブランチを checkout してきて rdflint のインタラクティブモードでクエリ投げる必要がある）

気軽にレビューに参加できるように、このような課題の解消方法を検討したい

[foooomio]

PR出てるブランチに対して，PR上からクエリ投げられるようになったりするとよいかもしれない。
（現状は、ブランチを checkout してきて rdflint のインタラクティブモードでクエリ投げる必要がある）

ふと思いついたんですけど、PR 上のコメントをトリガーとする GitHub Actions で rdflint を動かせば、それっぽいものが作れそうな気がしました。

イメージ→ https://twitter.com/nwtgck_ja/status/1235879073810771968

[takemikami]

クエリの結果をレビューア・レビューイに見てもらうケースではよさそうです。
ただ、クエリを試すたびにPullReqにコメントするのは、ちょっと気が引けるかも知れないですね。

[foooomio]

とりあえず動くものができたので公開してみました。→ foooomio/imasparql-playground#1

[foooomio]

あとこういうのも作ってみました。PR の diff が視覚的にわかりやすくなるかと思います。

foooomio/imasparql-playground#2

[foooomio]

Slack での提案に一定のいい反応が得られたので、現在 Open な PR にとりあえず手動でラベルを貼り付けてみました。

foooomio wrote:
あとは細かい改善案ですけど、たとえば 283.rdf が編集された PR には actions/labeler で「シャイニーカラーズ」のラベルを自動的に貼るとかすると、PR 一覧を見たときにレビューに参加しやすくなるのでは、と考えたんですがどうでしょう？

仮に actions/labeler で自動化するとしても、Live や CallTable のようにブランドで分割されていないものにはラベルを貼るのは難しいので、手動で対応することになりそうです。