nautilus-open-any-terminal icon indicating copy to clipboard operation
nautilus-open-any-terminal copied to clipboard

update super-linter 5 -> 6

Open Stunkymonkey opened this issue 1 year ago • 4 comments

update to the new CI workflow. see the checks.

Stunkymonkey avatar Feb 27 '24 21:02 Stunkymonkey

Is the new version of this any faster? I hate that it tries to check every single file in the repo with a "copy paste detector" and takes a horrendously long amount of time to even start. Can't we just run black/isort/pylint ourselves instead of relying on this?

lvxnull avatar Mar 09 '24 23:03 lvxnull

@lvxnull new version prints the status of each linter in a separate check.

Yes we could write this our own, and yes we would forget about new linters. So to be up-to-date with linters I think this is a nice way.

Stunkymonkey avatar Mar 10 '24 16:03 Stunkymonkey

The linters do not need to be constantly updated in order to be effective. I prefer not to spend 3 minutes waiting for a generic linter to finish checking files it does not actually understand.

If a file has been copy-pasted and passed manual code review, there is likely a valid reason for it. I do not like justifying my decisions to jscpd. Jscpd has minimal utility and depends on nodejs, which only lengthens the container startup speed.

I suggest sticking to the linters that are important to us (such as black, isort, pylint) and updating them as needed. We do not need to always be up to date just for the sake of having the latest version.

lvxnull avatar Mar 11 '24 17:03 lvxnull

rebased, but changed the last commit.

I totaly agree, that the linters do not need constant updates. What I really like about it: Not having to care about adding new linters for various things. But I guess we just have a different opinion on this.

If you want to we could add an additional black, pylint and isort to have faster feedback (not sure if this is in your mind; feels more like you hate the waiting-time).

Btw: The Linter is not a required Job (only the python build is). It only helps guiding.

Stunkymonkey avatar Mar 12 '24 20:03 Stunkymonkey