Skip to content
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

Consider using package.metadata to configure allow/warn/deny/forbid lint settings #537

Open
obi1kenobi opened this issue Sep 11, 2023 · 1 comment
Labels
C-enhancement Category: raise the bar on expectations E-help-wanted Call for participation: Help is requested to fix this issue. E-mentor Call for participation: Mentorship is available for this issue.

Comments

@obi1kenobi
Copy link
Owner

While we wait for external tools support for the lints table, consider using package.metadata to allow crates to configure their lint settings.

For example, this could address the tokio use case where they'd prefer to allow minor lints like #[must_use]. It would also allow us to add allow-by-default lints like "API got extended," which may be desirable for some use cases and not desirable in others.

Docs: https://doc.rust-lang.org/cargo/reference/manifest.html#the-metadata-table

@obi1kenobi obi1kenobi added C-enhancement Category: raise the bar on expectations E-help-wanted Call for participation: Help is requested to fix this issue. E-mentor Call for participation: Mentorship is available for this issue. labels Sep 11, 2023
@pksunkara
Copy link
Contributor

Related issue: #58

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: raise the bar on expectations E-help-wanted Call for participation: Help is requested to fix this issue. E-mentor Call for participation: Mentorship is available for this issue.
Projects
None yet
Development

No branches or pull requests

2 participants