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

Move clippy lints list to .cargo/config.toml #1394

Merged
merged 3 commits into from
Mar 21, 2022
Merged

Move clippy lints list to .cargo/config.toml #1394

merged 3 commits into from
Mar 21, 2022

Conversation

emilk
Copy link
Owner

@emilk emilk commented Mar 21, 2022

That way they apply to all crates equally.

See EmbarkStudios/rust-ecosystem#22 for why.

@emilk emilk merged commit fda8189 into master Mar 21, 2022
@emilk emilk deleted the clippy-in-config branch March 21, 2022 15:54
emilk added a commit that referenced this pull request Mar 22, 2022
emilk added a commit that referenced this pull request Mar 22, 2022
* Improve glow error reporting
* Add more check_for_gl_error calls
* Remove clippy lint list from egui_glow lib.rs
  - Forgotten in #1394
* egui_glow: move vao code to own file
* Cleanup: `use glow::HasContext as _;`

Co-authored-by: Zachary Kohnen <me@dusterthefirst.com>
@emilk emilk changed the title Move lints list to .carg/config.toml Move clippy lints list to .carg/config.toml Mar 31, 2022
@emilk emilk mentioned this pull request Mar 31, 2022
@emilk emilk changed the title Move clippy lints list to .carg/config.toml Move clippy lints list to .cargo/config.toml Mar 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant