-
Notifications
You must be signed in to change notification settings - Fork 118
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: cargo-denyを復活させ、mainブランチ上で動かす #855
chore: cargo-denyを復活させ、mainブランチ上で動かす #855
Conversation
752b9ae
to
4a11b3d
Compare
4a11b3d
to
04ecef1
Compare
❯ cargo deny check -s
advisories ok: 0 errors, 0 warnings, 0 notes
bans ok: 0 errors, 0 warnings, 76 notes
licenses ok: 0 errors, 0 warnings, 427 notes
sources ok: 0 errors, 0 warnings, 5 notes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
あ、このプルリクの意図をお聞きできれば🙏
ライセンス的に利用できないものを早めに検知できた方が良い、とかでしょうか?
(であればとても良いと思います!!!)
そうですね。あとは mainブランチでしか動かないようにしたのでマージ後にCIが落ちるといったことが発生しそうですが、一番ライブラリを出し入れするだろう私が手元で あ、でも「 @qryxip とRenovateからのPR」に対してだけ起動するという手もありそう。 |
なるほどです!! ライセンスチェックだけじゃなかったの忘れてました。 マージ後にCI落ちるのは一旦OKだと思います! レビューします! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!
一点だけ変更: |
これ↓については問題が起きてから考えることにして、マージしようと思います。
|
内容
題の通り。
関連 Issue
#333
その他
ドラフトにしているのは、
advisories
の領域だけunmaintain
だのvulnerability
が出ているためです。