search

os-climate / hazard

Onboarding, creation and transformation of climate hazard models for OS-Climate
Apache License 2.0
6 stars 13 forks source link

Configure ruff as linter and formatter. #113

Closed xbarra closed 1 month ago

xbarra commented 1 month ago

Co-authored-by: Arfima Dev dev@arfima.com Signed-off-by: Xavier Barrachina Civera xbarrachina@arfima.com

xbarra commented 1 month ago

Following the discussion in #111 I configured ruff as linter and formatter, removing the others.

I am commenting the markdown pre-commit until #94 is fixed

ciaransweet commented 1 month ago

Out of interest @xbarra is this PR rebased onto my commit? I'm surprised to see so many formatting changes, especially as my PR did a lot of it, I'd only expect a few files from the black/ruff differences

xbarra commented 1 month ago

Out of interest @xbarra is this PR rebased onto my commit? I'm surprised to see so many formatting changes, especially as my PR did a lot of it, I'd only expect a few files from the black/ruff differences

@ciaransweet It is rebased. The formatting changes are because the configuration in pyproject.toml adds several rules.

ciaransweet commented 1 month ago

Out of interest @xbarra is this PR rebased onto my commit? I'm surprised to see so many formatting changes, especially as my PR did a lot of it, I'd only expect a few files from the black/ruff differences

@ciaransweet It is rebased. The formatting changes are because the configuration in pyproject.toml adds several rules.

Gotcha, thanks!