Skip to content
This repository has been archived by the owner on Jul 16, 2021. It is now read-only.

Autoformat crate #198

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Autoformat crate #198

wants to merge 1 commit into from

Conversation

tiziano88
Copy link

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

This was done by running the `cargo fmt` command on the crate.
@AtheMathmo
Copy link
Owner

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.

@tiziano88
Copy link
Author

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!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants