Skip to content
This repository has been archived by the owner on Apr 12, 2023. It is now read-only.

DateTimeUtility.cs にMPL 2.0 のライセンス通知が含まれていない #229

Closed
b-wind opened this issue Jun 11, 2021 · 8 comments
Closed
Labels
confirmed 開発内部管理用 documentation 機能の改善や修正ではなく、ドキュメント類に関連するIssue

Comments

@b-wind
Copy link

b-wind commented Jun 11, 2021

不具合の内容 / Describe the bug

Covid19Radar/Common/DateTimeUtility.cs にMPL 2.0 のライセンス通知が含まれていません。

再現手順 / Steps to reproduce

以下を参照。

期待される挙動 / Expected behavior

MPL 2.0 のライセンス通知が追記される

スクリーンショット / Screenshots

なし。

動作環境 / Environments

すべて。

その他 / Additional context

リリース前に確認プロセスがあると望ましいかも知れません。


Internal IDs:

  • NFR 2704
@b-wind b-wind changed the title DateTimeUtility.cs にMPLv2のライセンス通知が含まれていない DateTimeUtility.cs にMPL 2.0 のライセンス通知が含まれていない Jun 11, 2021
@b-wind
Copy link
Author

b-wind commented Jun 12, 2021

#224 にて対応が含まれている模様(PRタイトルからは見つけられなかった)

@keiji keiji added the documentation 機能の改善や修正ではなく、ドキュメント類に関連するIssue label Jun 12, 2021
@b-wind
Copy link
Author

b-wind commented Jun 12, 2021

GitHub ベースで考えるなら、push 時にライセンス付与してくれる Action 等はあります。
https://github.com/marketplace/actions/add-license-action
これはファイル単位じゃなかった。

ライセンス通知が含まれてないとpush出来ない方が良いのかは悩み所。

どちらにせよ、各 Store にリリースされた後にGitHubに反映という運用スタイルだと防げそうにないですね。

@Meiryo7743
Copy link
Contributor

Meiryo7743 commented Jun 12, 2021

今後のためにも,MPL ライセンスの記載漏れの事故が生じない仕組みを検討する必要がありますね……。

いま,内部リポジトリと GitHub 上での公開リポジトリの 2 つが存在し,定期的な同期がなされるわけです。したがって,リポジトリの置かれる環境に依存しない方法でチェックできるのが理想的と考えられます。

上記を踏まえ,Git hooks を導入してみるのはどうでしょうか。元々 Git に組み込まれている機能で,例えば clone や commit 時に任意の処理が走るというものです。

さらに念を入れるのでしたら,それこそ CI でもチェックする仕組みを構築するのが良いでしょう。

cf. https://git-scm.com/docs/githooks

@b-wind
Copy link
Author

b-wind commented Jun 13, 2021

みんな考える事は似てるなぁという感じで見てる。

https://github.com/google/addlicense

@b-wind
Copy link
Author

b-wind commented Jun 13, 2021

無条件に MPL 2.0 にすると Xamarin.ExposureNotification とかで困りそう。
(元がMITだし、リライセンス出来そうではある)

@b-wind
Copy link
Author

b-wind commented Jun 25, 2021

基本的にこのIssueでは該当ファイルにどのような方法でも良いので追加されることを目標としています。
ライセンスヘッダ付与の自動化を考えている方々は別途その趣旨の Issue を立てて貰った方が良い様に思います。

@Takym
Copy link
Contributor

Takym commented Jun 26, 2021

#255 にて建てました。

追記

Issue 番号を間違えていました。すみません。

@keiji
Copy link
Collaborator

keiji commented Jan 8, 2022

c496a3b が developに取り込まれたのでcloseします。

@keiji keiji closed this as completed Jan 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
confirmed 開発内部管理用 documentation 機能の改善や修正ではなく、ドキュメント類に関連するIssue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants