Skip to content

辞書を push する前の前処理 #2

Open

Description

@yutopia

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

Activity

  1. conao3 commented on Oct 1, 2021

    @conao3
    Member

    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

  2. conao3 commented on Oct 1, 2021

    @conao3
    Member

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

  3. added a commit that references this issue on Oct 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions