search

OSGeo / grass

GRASS GIS - free and open-source geospatial processing engine
https://grass.osgeo.org
Other
791 stars 288 forks source link

style: Fixes some unnecessary-lambda (PLW0108) #3956

Closed echoix closed 1 day ago

echoix commented 2 days ago

Concerns Pylint rule "unnecessary-lambda / W0108"

Using ruff check --output-format=concise --select PLW0108 --preview --fix --unsafe-fixes.

Didn't apply to cases where it wasn't clear if it was safe.

Part of the effort to introduce Pylint 3.x for https://github.com/OSGeo/grass/issues/3921

Uses the fixes provided for ruff rule unnecessary-lambda (PLW0108) to fix part of Pylint's "unnecessary-lambda / W0108".

echoix commented 1 day ago

To help reviewing, it is useful to know that some of the changes happen to align to a pattern already used inside the same file, like for the "connect()" functions, like here: https://github.com/OSGeo/grass/pull/3956/files#diff-d14bd7d2a6d585d27253c5e579b216708d09a8f4c063c565980e5e7194f5993aR208-R209