feather-rs / feather

A Minecraft server implementation in Rust
Apache License 2.0
2.61k stars 142 forks source link

Use cargo deny in CI #538

Open Plecra opened 2 years ago

Plecra commented 2 years ago

Status

Description

As mentioned in https://github.com/feather-rs/feather/issues/531#issuecomment-1032325505, I've setup the repo to use cargo deny. It is now correctly checking everything, and allowing for a set of exceptions in deny.toml.

All that's left is adding a workflow for running the tool.

Related issues

Leave empty if none

Checklist

Note: if you locally don't get any errors, but GitHub Actions fails (especially at clippy) you might want to check your rust toolchain version. You can then feel free to fix these warnings/errors in your PR.

Iaiao commented 2 years ago

quill is licensed under Apache 2.0 or MIT

Plecra commented 2 years ago

Right, I was surprised about that :) I couldn't find an explicit mention of the MIT license in the repo - where is Apache+MIT declared?

Iaiao commented 2 years ago

There are quill/LICENSE-APACHE and quill/LICENSE-MIT. There's no explicit mention of dual licensing in README.md and Cargo.toml though, so I'm not sure about that. @caelunshun is quill dual licensed, or why is there 2 license files?