-
-
Notifications
You must be signed in to change notification settings - Fork 316
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
style: Fixes some unnecessary-lambda (PLW0108) #3956
Conversation
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
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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not too familiar with lambda expressions, but changes look consistent and according to style-guide.
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
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
This reverts commit 0cc7cbf.
…3956)" (OSGeo#4127) * Revert "style: Fixes some unnecessary-lambda (PLW0108) (OSGeo#3956)" This reverts commit 0cc7cbf. * ruff: Ignore unnecessary-lambda (PLW0108) in gui
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 #3921
Uses the fixes provided for ruff rule unnecessary-lambda (PLW0108) to fix part of Pylint's "unnecessary-lambda / W0108".