automl / neps

Neural Pipeline Search (NePS): Helps deep learning experts find the best neural pipeline.
https://automl.github.io/neps/
Apache License 2.0
39 stars 11 forks source link

[CI] Migration to ruff #77

Closed eddiebergman closed 2 months ago

eddiebergman commented 2 months ago

We talked about migrating most CI tools to just be ruff for several reasons, namely it superseedes many existing tools, does so much faster and also implements many auto-fixes.

To go about it, I would propose to do it file by file or directory by directory, in such a way as to not cause conflicts with on-going work.

A quick run of ruff with the setup of rules I use for AMLTK found 499 issues with 291 of them being auto-fixable.

Found 499 errors.
[*] 291 fixable with the `--fix` option.
eddiebergman commented 2 months ago

I've heard that pre-commit formatting is running on the docs/ folder, this should be fixed