search

jabacat / jml

JABACAT-created machine learning library from scratch.
5 stars 5 forks source link

Create format.yml #50

Closed JakeRoggenbuck closed 10 months ago

JakeRoggenbuck commented 11 months ago

fixes #28

JakeRoggenbuck commented 11 months ago

I don't know if this is possible, but is there a way to set it so that it enforces type& variable instead of type &variable? It seems the former is already fairly consistent in our codebase and this would change it.

I can take a look

adamhutchings commented 11 months ago

@Sophon96 @JakeRoggenbuck what remains?

JakeRoggenbuck commented 11 months ago

I fixed the reference align left, clang-format config, and the unneeded flag in the command to run the workflow formatting

JakeRoggenbuck commented 11 months ago

I fixed the reference align left, clang-format config, and the unneeded flag in the command to run the workflow formatting

Anything else @adamhutchings @Sophon96 ?

JakeRoggenbuck commented 10 months ago

That broke something. I will reset to e944bd3

JakeRoggenbuck commented 10 months ago

Force pushed the two edits to format check out

JakeRoggenbuck commented 10 months ago

Could I please get a review for this?