Format this repo and check formatting on pushs #40

Closed

cactusdualcore wants to merge 2 commits from (deleted):main into main

cactusdualcore commented

2025-12-07 20:27:35 +01:00

First-time contributor

This PR is two commits.

The first introduces the format checking on push.

The second formats the relevant crates, so that they pass CI. I am sorry for the extent of this commit. It touches most Rust files in this repo. It will be bad pretty annoying to review. But your code will be formatted and (hopefully) more readable for everyone.

Fixes thetxt/oxide#35

This PR is two commits. The first introduces the format checking on push. The second formats the relevant crates, so that they pass CI. I am sorry for the extent of this commit. It touches most Rust files in this repo. It will be bad pretty annoying to review. But your code will be formatted and (hopefully) more readable for everyone. Fixes thetxt/oxide#35

cactusdualcore added 2 commits

2025-12-07 20:27:35 +01:00

ci: check formatting on push b7a9e525ed

This commit reformates the main.yml file and introduces the
'rustfmt_check' job to ensure all rust files in PRs to the 'server',
'lib', 'proxy' and 'data_generator_tools' crates are properly formatted.
To avoid spamming the CI logs, the command runs with the --quiet flag.

style: run 'cargo fmt' eb05ba7003

cactusdualcore commented

2025-12-07 20:29:35 +01:00

Author

First-time contributor

Btw sorry for reformatting the main.yml file. That was an accident.

Btw sorry for reformatting the `main.yml` file. That was an accident.

thetxt commented

2025-12-08 18:20:04 +01:00

Owner

Its just my personal preference and its probably deeply offending to most people in the rust community, but I dont like autoformatters like rustfmt. I feel like sometimes its more readable to have things formatted the "wrong" way. Again, just how my brain works.
Maybe in the far future I will revisit this and come up with a rustfmt config that more closely aligns to my personal preferences.

Its just my personal preference and its probably deeply offending to most people in the rust community, but I dont like autoformatters like rustfmt. I feel like sometimes its more readable to have things formatted the "wrong" way. Again, just how my brain works. Maybe in the far future I will revisit this and come up with a rustfmt config that more closely aligns to my personal preferences.