search

OliverFM / rust_light

A tensor processing library in rust.
MIT License
3 stars 0 forks source link

Add clippy checks on push #35

Closed aidenfoxivey closed 1 year ago

OliverFM commented 1 year ago

Thanks for adding this!

aidenfoxivey commented 1 year ago

I think it's probably advantageous to separate this to a different job?

OliverFM commented 1 year ago

I think it's probably advantageous to separate this to a different job?

yeah maybe the best way to do this is to disable all the warnings that the code currently produces, then fix them as we go along. Otherwise this will probably never be merged

aidenfoxivey commented 1 year ago

Fair enough, that's a good perspective. I'll modify and submit by EOD.

aidenfoxivey commented 1 year ago

This should probably work? I hope at least. Let me know if there are any issues here.

OliverFM commented 1 year ago

@aidenfoxivey this looks great! I will merge this now, and we can make it a required check once I have fixed the remaining lint issues!