-
Notifications
You must be signed in to change notification settings - Fork 13.6k
Closed
Labels
A-licensingArea: Compiler licensingArea: Compiler licensingA-metaArea: Issues & PRs about the rust-lang/rust repository itselfArea: Issues & PRs about the rust-lang/rust repository itselfT-leadership-councilRelevant to the leadership council.Relevant to the leadership council.
Description
License validation (https://github.com/rust-lang/rust/blob/master/src/tools/tidy/src/deps.rs) for rust-analyzer is currently not being done. I think it would probably be good to include it in the license checks, just to double-check that something inappropriate doesn't slip in.
I manually checked them, and they seem ok at this point. It would be helpful if the ra crates themselves included a license field in their Cargo.toml for automatic validation.
cc @matklad
matklad and simonvandel
Metadata
Metadata
Assignees
Labels
A-licensingArea: Compiler licensingArea: Compiler licensingA-metaArea: Issues & PRs about the rust-lang/rust repository itselfArea: Issues & PRs about the rust-lang/rust repository itselfT-leadership-councilRelevant to the leadership council.Relevant to the leadership council.