Skip to content

Add documentation about RUSTFMT environment variable #4426

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.

Sign up for GitHub

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

Jump to bottom
Closed
2 tasks done
topecongiro opened this issue Sep 16, 2020 · 4 comments · Fixed by #4464
Closed
2 tasks done

Add documentation about RUSTFMT environment variable #4426

topecongiro opened this issue Sep 16, 2020 · 4 comments · Fixed by #4464
Labels
documentation good first issue Issues up for grabs, also good candidates for new rustfmt contributors hacktoberfest help wanted

Comments

@topecongiro
Copy link
Contributor

topecongiro commented Sep 16, 2020 •
edited by calebcartwright
Loading

  • Update README.md
  • Update cargo's documentation. We should do this after we release the version with RUSTFMT support.

Related: #4419
@topecongiro topecongiro added good first issue Issues up for grabs, also good candidates for new rustfmt contributors help wanted documentation labels Sep 16, 2020
@MonliH
Copy link
Contributor

MonliH commented Oct 1, 2020

When is the new version going to be released? When it does, I could help.

@calebcartwright
Copy link
Member

When is the new version going to be released?

Have to get a few other PRs landed in other repos since the next version contains a bump of the rustc-ap-* crates. Will update here once we have a more concrete answer

When it does, I could help.

Fantastic, thank you!

@calebcartwright
Copy link
Member

rustfmt 1.4.22 has landed on nightly (and I believe beta now as well) so this should be good to go @MonliH if you are still interested in working on it

@MonliH
Copy link
Contributor

MonliH commented Oct 10, 2020

Yep, sure! Will do.

This was referenced Oct 10, 2020
Merged
Merged
bors added a commit to rust-lang/cargo that referenced this issue Oct 13, 2020
@bors
Auto merge of #8767 - MonliH:master, r=ehuss
31e1f1f 
Document RUSTFMT environment variable

This PR documents the `RUSTFMT` enviorment variable, as specified in [rust-lang/rustfmt/4426](rust-lang/rustfmt#4426) and [rust-lang/rustfmt/4419](rust-lang/rustfmt#4419).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
documentation good first issue Issues up for grabs, also good candidates for new rustfmt contributors hacktoberfest help wanted
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Document RUSTFMT environment variable
3 participants
@calebcartwright @topecongiro @MonliH