-
Notifications
You must be signed in to change notification settings - Fork 111
ライセンス通知の自動付与ツールを作成する #255
Comments
ありがとうございます。 全部自動処理する前の段階として、CIで「ライセンスが追加されていないファイルが追加されていたらエラーにしすることでマージされるのを防ぐ」手当かなと思います。これなら内部リポジトリからのSyncをPull Requestという扱いでチェックするようにすれば、DateTimeUtility.csのようなことが起きる可能性は小さくなると考えていますが、いかがでしょうか。 そういう意味ではこのIssueとも話がずれるわけですが…ひとまず2つのPull Request( #236 #237 )でGitHub Actionsで実行できて、ライセンス通知がないファイルの「追加」を検出できるものはありそうでしょうか。特に #237 は
という記述があるので、そういった使われ方を想定しているのかなと思っています。 |
確かに,その頻度はあまり高くありませんから,わざわざ pre-commit でチェックするまでもないと言えそうです。 #237 は将来的に GitHub Actions 等の CI で実行することも想定しています。 イメージとしては,「Create Pull Request」と組み合わせることで,[スクリプトを実行]→[Base branch に対し,通知漏れの修正 PR を自動作成する]……といった具合です。よろしければ別途 PR を作成いたしましょうか……? |
ライセンス通知が付与していないファイルの検出であればより単純な機構で動くと思います。 #236 は手動で動かす事を前提に実装したため、標準出力にログを書き込んでいます。 |
@Meiryo7743 @Takym 検出の機能をまずはGitHub Actionsに組み込む。問題なく動けばPull Requestに対するコメントであったり、 @Meiryo7743 さん提案のPull Reqeustに対して追加でPull Requestを出すなどができそうですね。 |
|
はい。よろしくお願いします。 |
変更しました! |
その機能リクエストは何らかの問題に関連しますか
本リポジトリではファイルの先頭にライセンス通知を記述する決まり(#80)になっていますが、書き忘れてしまう事も少々あります。例えば DateTimeUtility.cs (#229) 等があります。
cocoa/Covid19Radar/Covid19Radar/Common/DateTimeUtility.cs
Lines 1 to 18 in b185621
解決策についてお書きください
#236 (Windows版)、#237 (Linux版) により既に解決策となる PR が作成されています。
あなたが考える代替案についてご説明ください
今のところは特にありません。
その他
IDateTimeUtility
をサービス化 #224 で既に解決しています。Internal IDs:
The text was updated successfully, but these errors were encountered: