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

Cargo fmt #2009

Merged
merged 2 commits into from
Nov 14, 2022
Merged

Cargo fmt #2009

merged 2 commits into from
Nov 14, 2022

Conversation

sgued
Copy link
Contributor

@sgued sgued commented Oct 30, 2022

  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

  • Are you doing the PR on the next branch?

This PR runs cargo fmt and adds cargo fmt --check to the pipeline to prevent future PRs from being merged without proper formatting in the future. This is something that has happened in the past:

Having everything be formatted always avoids having unexpectedly large diffs when someone uses format-on-save and doesn't turn it off.

The formatting was not that far off from what cargo fmt gives us so it shouldn't cause that many conflicts.

@sgued sgued mentioned this pull request Oct 30, 2022
2 tasks
@Keats Keats merged commit 697e715 into getzola:next Nov 14, 2022
Keats pushed a commit that referenced this pull request Feb 16, 2023
* Run cargo fmt

* Add cargo fmt --check to pipeline
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