Issue 27/configuring lint levels #53
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Places where `single_use_lifetimes`-forbidden code was present were modified to follow the new guidelines.
Code that was invalidated by the new guidelines was changed to respect them.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Resolves #27.
Resolves #32, Resolves #33, Resolves #35, Resolves #36, Resolves #37, Resolves #38, Resolves #42, Resolves #43.
Moves noop_method_call from allow to deny.
Moves single_use_lifetimes from allow to deny.
Moves unreachable_pub from allow to deny.
Moves unsafe_code from allow to deny.
Moves unsafe-op-in-unsafe-code from allow to deny.
Moves unused_import_braces from allow to deny.
Moves unused_lifetimes from allow to deny.
Moves incomplete_features from warn to allow.
Denies all warnings in the crate.
All code that was invalidated by the changes was corrected to be aligned to the new guidelines.
#30 remains to be completed. As it depends on #31, which might require a lot of time, it will be completed on its own and not as part of this merge request and #27.