gluonts icon indicating copy to clipboard operation
gluonts copied to clipboard

ci: update lints Ruff & docformatter

Open Borda opened this issue 1 year ago • 2 comments

Issue #, if available: dofformatter gave a false positive signal; it has lots of issues, but as --check was missing, it just fixed without any real code change and so passed green even if it failed, see for example https://github.com/awslabs/gluonts/actions/runs/7975893111/job/21775154024?pr=3130

Description of changes:

validation of the actual Ruff & Docfroatter on the latest codebase

Please tag this pr with at least one of these labels to make our release process faster: this is just fixing the actual linting issue, but the better and ext step shall be #3111

cc: @jaheba @kashif @lostella

Borda avatar Feb 20 '24 15:02 Borda

I do think that the stile check is related, but anyway would be good to update Black (sending PR #3131) so we better see if it is related :chipmunk:

Borda avatar Feb 20 '24 16:02 Borda

not sure what Black does not like:

would reformat /home/runner/work/gluonts/gluonts/src/gluonts/maybe.py
would reformat /home/runner/work/gluonts/gluonts/src/gluonts/mx/block/snmlp.py
would reformat /home/runner/work/gluonts/gluonts/src/gluonts/nursery/tsbench/src/tsbench/analysis/utils/misc.py
would reformat /home/runner/work/gluonts/gluonts/src/gluonts/torch/util.py

Borda avatar Feb 20 '24 17:02 Borda

@Borda unrelated, can you kindly send me an email [email protected] I wanted to ask you something about lightning?

kashif avatar Feb 26 '24 08:02 kashif

@kashif @lostella resolved conflicts, could you pls check it :flamingo:

it would also need to update the required checks, remove flake8 and add lint (ruff) instead

Borda avatar Apr 05 '24 21:04 Borda

@lostella, thank you for your detailed read and for pointing out all the issues... I will try to address them just to clarify, the docformatter even was running was ignored, so this is just trying to re-enable it properly :)

I think that Ruff's rule D could help here

Borda avatar Apr 12 '24 20:04 Borda

@lostella, do you have any suggestions on what we should do with the remaining cases? :flamingo:

Borda avatar Apr 18 '24 08:04 Borda

@Borda just one comment then this can go in 🚀

lostella avatar Apr 19 '24 12:04 lostella

@lostella committed your suggestion, thx :)

Borda avatar Apr 19 '24 12:04 Borda

@lostella hope all is fine now, and with this resolved, I could finish the other PR :flamingo:

could you pls replace flake8 with Ruff & Docformat in the required checks?

Borda avatar Apr 25 '24 12:04 Borda

@lostella, how are you doing? mind have look :)

Borda avatar May 07 '24 11:05 Borda