Skip to content

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

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants