rusty-machine icon indicating copy to clipboard operation
rusty-machine copied to clipboard

Published 20 hours ago • verified publisher icon AtheMathmo

Autoformat crate

Open tiziano88 opened this issue 6 years ago • 2 comments

This was done by running the cargo fmt command on the crate.

tiziano88 avatar Nov 30 '18 21:11 tiziano88

This looks fine to me (but it is a lot of code to check over).

It looks like the library may be getting some actual maintenance soon ( #199 ) so if that involves some refactor I'll leave it up to those parties to decide whether these formatting improvements should come before or after.

AtheMathmo avatar Dec 07 '18 22:12 AtheMathmo

Thanks for the reply! The format PR was in fact a prerequisite for more refactorings that I would like to send to you soon, if there is interest. I think you should check the code, or run the same command yourself and check there are no extraneous diffs, it should be a quick one. Anyway let me know what you prefer, I would like it for rusty-machine to become the de facto ML crate in Rust!

tiziano88 avatar Dec 09 '18 12:12 tiziano88