辞書を push する前の前処理

[yutopia]

SKK-JISYO は登録する際に整形する必要がありますが、GitHubにpushする前に整形が行われるべきである。
push 操作のときに整形ツールを hook できる仕組みは無いか？

[JunShiozawa]

yutopiaさま
はじめまして。

push 操作のときに整形ツールを hook できる仕組みは無いか？

「push 操作のとき」
という条件なら
そのような仕組みは
無いでしょう。
そもそも
$ git commit
で確定させたリビジョンを
$ git push
するのがGitの前提ですから。

GitHubにpushする前に整形が行われるべきである。

整形の処理が必要であるならば
整形の処理のタイミングは
「pushする前」
つまり
$ git push
する前
のタイミング
ではなく
$ git add
する前のタイミングで整形の処理が行われるのがGitのバージョン管理の前提かと思います。

「push 操作のとき」に整形するのは無理です。

プルリクエストがあった際に
プルリク審査担当者によって(プルリク審査担当者が何らかの整形済みチェックツールを使用する行為も含めて)整形済みかをチェックする(つまり貢献者からプルリクエストされた辞書ファイルが未整形で辞書ファイルが壊れていないかの確認をプルリク審査担当者が行う)のがGitやGithubでの共同開発の一般的なルールかと思います。

yutopiaさまは
「push 操作のとき」にhookしたい何らかの既製の整形ツールをすでにイメージされているのかもしれませんが
整形の処理のタイミングが
$ git add
する前
という前提での
お話になりますが
$ git add
する前に
yutopiaさまが
すでにイメージされている何らかの既製の整形ツールを
使用して整形するか
または
自作のシェルスクリプトで整形する
または
自作のPython等のスクリプトで整形する
または
SKK-JISYOの整形の目的にどうしても必要な処理速度かは微妙ですが
自作のC言語の整形プログラムで整形する
などの
アプローチが考えられます。

yutopiaさま
これからもよろしくお願いいたします。

[conao3]

pre-push フックは、 git push を実行した際、リモート参照が更新された後、オブジェクトの転送が始まる前に実行されます。 このフックはリモートの名前と場所を引数に取ります。また、これから更新する参照のリストを stdin から受け取ります。 このフックは、プッシュを行う前に、更新される参照を検査するのに使用できます（ゼロ以外の値を返すとプッシュが中断されます）。

今はそのものズバリで pre-push hookがあるようです。

https://git-scm.com/book/ja/v2/Git-%E3%81%AE%E3%82%AB%E3%82%B9%E3%82%BF%E3%83%9E%E3%82%A4%E3%82%BA-Git-%E3%83%95%E3%83%83%E3%82%AF

[conao3]

実際のところはCIでチェックして、そのCIがpassしないとmergeできないようにするというのが良さそうですが。

[JunShiozawa]

conao3さま
はじめまして。

今はそのものズバリで pre-push hookがあるようです。
実際のところはCIでチェックして、そのCIがpassしないとmergeできないようにするというのが良さそうですが。

プルリク審査補助の便利な自動化ツールあるんですね(⌒▽⌒)

conao3さま
これからもよろしくご指導お願いいたします。