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

add rust-lang/rust#50723 as an easy-level call for participation #685

Merged
merged 1 commit into from
Aug 2, 2018

Conversation

zackmdavis
Copy link
Member

@zackmdavis zackmdavis commented Aug 2, 2018

rust-lang/rust#50723 is a good beginner issue, providing familiarity with the rustc build system while accomplishing a chore with real importance.

@nasa42 nasa42 merged commit 1fd5e41 into rust-lang:master Aug 2, 2018
@zackmdavis zackmdavis deleted the applicability_rush branch August 3, 2018 03:18
kennytm added a commit to kennytm/rust that referenced this pull request Aug 4, 2018
…ebank

App-lint-cability

@eminence recently pointed out (rust-lang/cargo#5846) that it's
surprising that `cargo fix` (now shipping with Cargo itself!) doesn't
fix very common lint warnings, which is as good of a reminder as any
that we should finish rust-lang#50723.

(Previously, we did this on the librustc and libsyntax crates in rust-lang#50724. I filed rust-lang/this-week-in-rust#685 in hopes of recruiting new contributors to do the rest.)

r? @estebank
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.

2 participants