Skip to content

Update rustfmt #864

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

Merged
merged 1 commit into from
Jun 8, 2023
Merged

Update rustfmt #864

merged 1 commit into from
Jun 8, 2023

Conversation

ali-behjati
Copy link
Collaborator

@ali-behjati ali-behjati commented Jun 8, 2023

This PR removes an unused rustfmt and adds edition = 2021 to the global rustfmt.

Rustfmt uses a default of edition = 2015 and when we run it it won't be able to format the codes (as some syntaxes and codes are unsupported for 2015 edition) and Vim's formatter uses it. This won't happen in cargo fmt because it will fetch the correct edition from Cargo.toml.

See here for more info.

@ali-behjati ali-behjati requested a review from Reisen June 8, 2023 11:45
@vercel
Copy link

vercel bot commented Jun 8, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Ignored Deployments
Name Status Preview Updated (UTC)
example-oracle-amm ⬜️ Ignored (Inspect) Jun 8, 2023 11:45am
xc-admin-frontend ⬜️ Ignored (Inspect) Jun 8, 2023 11:45am

@ali-behjati ali-behjati merged commit 7847a79 into main Jun 8, 2023
@ali-behjati ali-behjati deleted the update-rustfmt branch June 8, 2023 13:07
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