papis icon indicating copy to clipboard operation
papis copied to clipboard

ci: use ruff for linting

Open kiike opened this issue 2 years ago • 4 comments

HI again!

Here is a companion PR for the discussion in #812.

It adds linting with ruff, which catches many more style "issues". Along getting rid of many dependencies, it provides faster linting and formatting, which can be pretty cool when used with pre-commit, for instance.

I tried to port some of the excluded rules.

WIP

kiike avatar Apr 12 '24 10:04 kiike

I'm on board with switching to ruff, but probably after we make a release with the current stuff. It's definitely due :grin:

alexfikl avatar Apr 12 '24 11:04 alexfikl

Sure. It can add a lot of new rules, so that will involve picking which ones the project wants. Luckily, it auto-fixes most of the minor issues.

kiike avatar Apr 12 '24 13:04 kiike

I'm definitely on board with using ruff. I've been using it on my side for a while and it would be nice to have some way of easily enforcing a nice consistent style :).

jghauser avatar Apr 13 '24 09:04 jghauser

Cool! We'll keep this PR on the backburner!

kiike avatar Apr 13 '24 13:04 kiike