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.

Sign up for GitHub

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

Jump to bottom

Cleanup lint management #59

Open
scott-ainsworth opened this issue Dec 21, 2021 · 0 comments
Open

Cleanup lint management #59

scott-ainsworth opened this issue Dec 21, 2021 · 0 comments
Assignees
@scott-ainsworth
Labels
maint Technical change required for routine maintenance or that enhances maintainability. Technical debt.
Milestone
Version 1.1

Comments

@scott-ainsworth
Copy link
Owner

Currently, lints are handled using the nightly version of Rust and enables lints only available in the nightly version. Additionally, many normally-allowed lints are enabled by adding #![warn(...)] lines to the source (which is ugly).

Requirements

  • Enable lints in .cargo/config.toml.
  • Cleanup code to eliminate lints.
scott-ainsworth added a commit that referenced this issue Dec 21, 2021
@scott-ainsworth
Cleanup lints and lint management
28999f1 
Issue #59
@scott-ainsworth scott-ainsworth added the maint Technical change required for routine maintenance or that enhances maintainability. Technical debt. label Dec 22, 2021
@scott-ainsworth scott-ainsworth added this to the Version 1.1 milestone Dec 22, 2021
@scott-ainsworth scott-ainsworth self-assigned this Dec 22, 2021
This was referenced Dec 22, 2021
Open
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@scott-ainsworth scott-ainsworth

Labels
maint Technical change required for routine maintenance or that enhances maintainability. Technical debt.
Projects
None yet
Milestone
Version 1.1
Development

No branches or pull requests
1 participant
@scott-ainsworth