Skip to content
This repository has been archived by the owner on Nov 24, 2023. It is now read-only.

README update aiming to clarify things for less experienced rustaceans #222

Merged
merged 3 commits into from
Oct 16, 2023

Conversation

3tilley
Copy link
Contributor

@3tilley 3tilley commented Oct 15, 2023

Hello,

Following on from #221 I thought I would have a go at improving the README a little. I've looked at this crate several times in the past thinking it did something different to what it actually did!

Please feel free to correct anything I've put that isn't correct. The goal is to let people know what the library does, as well as directing them that what they're looking for probably isn't here.

Max

Copy link
Member

@weihanglo weihanglo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution!

cc @Muscraft. I know you are highly interested in auto fixing stuff.

Readme.md Outdated Show resolved Hide resolved
- `rustfix`, a library for consuming and applying suggestions in the format that `rustc` outputs
- and `cargo-fix`, a binary that works as cargo subcommand and that end users will use to fix their code.
- `rustfix`, a library for consuming and applying suggestions in the format that `rustc` outputs (this crate)
- `cargo-fix`, a binary that works as cargo subcommand and that end users will use to fix their code (maintained in the [cargo](https://github.com/rust-lang/cargo/blob/master/src/cargo/ops/fix.rs) repo).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: I may link to https://github.com/rust-lang/cargo/blob/master/src/bin/cargo/commands/fix.rs, which is more like the entry of the entire "binary" part of cargo-fix.

Readme.md Outdated Show resolved Hide resolved
Manishearth and others added 2 commits October 15, 2023 19:51
Co-authored-by: Weihang Lo <weihanglo@users.noreply.github.com>
Co-authored-by: Weihang Lo <weihanglo@users.noreply.github.com>
Copy link
Member

@Manishearth Manishearth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

@Manishearth Manishearth merged commit 10fd5eb into rust-lang:master Oct 16, 2023
3 checks passed
3tilley added a commit to 3tilley/rustfix that referenced this pull request Oct 17, 2023
This was referenced Oct 17, 2023
Manishearth added a commit that referenced this pull request Oct 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants