Allow banning multiple versions of a specific crate #365
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds a
multiple-versions
boolean field to the deny list in bans, allowing themultiple-versions
lint level to be set to "allow" or "warn" for all crates and overriding it to "deny" for a chosen few:Settings both
multiple-versions
andwrappers
is explicitely not supported as I don't think it would make sense, I added an error diagnostic for that specific case that unfortunately required to add a fewOption
andSpanned
wrappers inCrateBan
to display the relevant labels. The exact format of the configuration could be different anyway, I simply went with an option that seemed to require only a few changes instead of introducing a completely new deny list specifically for duplicate crates for instance