ultimate_rust_crash_course icon indicating copy to clipboard operation
ultimate_rust_crash_course copied to clipboard

rustfmt source code and GitHub action

Open cpmachado opened this issue 11 months ago • 2 comments

Before anything, I'm sorry for #47 and #46 , they were mistakes.

On something useful, I'm suggesting, as referred on HowToLearnRust.md:66, to use rustfmt.

I like tooling and rules to follow, and the means to follow them.

As part of my apology, I wrote this github action largely adapted from : https://github.com/actions-rust-lang/setup-rust-toolchain

On another note, I'm loving your crash course and will recommend. I'm in the middle, but loved to discover clippy, which was referenced in one of the exercises.

Best regards, cpmachado

cpmachado avatar Mar 02 '25 01:03 cpmachado

It had a bug, sorry, now all good: https://github.com/cpmachado/ultimate_rust_crash_course/actions/runs/13610721051/job/38047520433

cpmachado avatar Mar 02 '25 01:03 cpmachado

As I go along I'd be interested in doing some testing for exercises.

I've noticed it isn't covered by the current course, but when I get to next one, would you be against the idea of adding some tests for this?

cpmachado avatar Mar 02 '25 01:03 cpmachado

I'm so glad you loved my crash course!

Apologies for the delay in my response. Life has been crazy for me this year.

Unfortunately, there are a lot of conflicts with the comprehensive update I made and your proposed changes, so I'm going to close this pull request. Using rustfmt and clippy are both great practices that I highly recommend for real code!

CleanCut avatar Oct 14 '25 19:10 CleanCut

thanks for the consideration :-)

cpmachado avatar Oct 24 '25 09:10 cpmachado

No problem!

CleanCut avatar Oct 24 '25 16:10 CleanCut