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

Update rustfmt with 2021 edition #2196

Merged
merged 2 commits into from
Dec 7, 2021

Conversation

yanganto
Copy link
Contributor

Update Rust formatting tool with 2021 edition

Proposed Changes

  1. Update README
  2. Add flag for 2021 edition of Rustfmt

Readiness Checklist

Author/Contributor

  • If documentation is needed for this change, has that been included in this pull request

Reviewing Maintainer

  • Label as breaking if this is a large fundamental change
  • Label as either automation, bug, documentation, enhancement, infrastructure, or performance

@azure-pipelines
Copy link

There was an error handling pipeline event 90b5a28d-a722-4258-9e3d-82312e98377d.

@yanganto
Copy link
Contributor Author

yanganto commented Dec 7, 2021

How can I see the error handling pipeline event?

@admiralAwkbar
Copy link
Collaborator

@yanganto Thanks for the add!

@admiralAwkbar admiralAwkbar merged commit 1d4680e into super-linter:main Dec 7, 2021
@ferrarimarco
Copy link
Collaborator

There are no tests in this PR, but we didn't notice :( That's why builds are failing now.

@yanganto
Copy link
Contributor Author

yanganto commented Dec 8, 2021

@ferrarimarco This PR just provides new edition option for rustfmt, should I add test for this? If so, please kindly tell me, I am willing to help on this.

@yanganto yanganto deleted the rust-2021-edition branch December 8, 2021 02:19
@ferrarimarco
Copy link
Collaborator

Hi @yanganto! No worries, I added them (plus a fix) in #2215

sarahc23 pushed a commit to 23andMe/super-linter that referenced this pull request May 6, 2022
sarahc23 pushed a commit to 23andMe/super-linter that referenced this pull request May 6, 2022
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.

None yet

3 participants