Skip to content
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

Increase the usage of augmented assignment statements #5997

Closed
elfring opened this issue Oct 30, 2021 · 2 comments
Closed

Increase the usage of augmented assignment statements #5997

elfring opened this issue Oct 30, 2021 · 2 comments
Labels
⏩ type: Enhancement Improve existing features

Comments

@elfring
Copy link

elfring commented Oct 30, 2021

👀 Some source code analysis tools can help to find opportunities for improving software components.
💭 I propose to increase the usage of augmented assignment statements accordingly.

Would you like to integrate anything from a transformation result which can be generated by a command like the following?
(:point_right: Please check also for questionable change suggestions because of an evolving search pattern.)

[Markus_Elfring@fedora lokal]$ perl -p -i.orig -0777 -e 's/^(?<indentation>\s+)(?<target>\S+)\s*=\s*\k<target>[ \t]*(?<operator>[+\-%&|^@]|\*\*?|\/\/?|<<|>>)/$+{indentation}$+{target} $+{operator}=/gm' $(find ~/Projekte/scikit-image/lokal -name '*.py')
@mkcor
Copy link
Member

mkcor commented Nov 6, 2021

@rfezzani should this issue be converted into a discussion?

@rfezzani rfezzani added the ⏩ type: Enhancement Improve existing features label Nov 6, 2021
@rfezzani
Copy link
Member

rfezzani commented Nov 6, 2021

Thank you @elfring for this suggestion and sorry for only answering after a week 🙏
You are right @mkcor, following the idea of dedicating Github issues to bugs and Github discussion for any other request, I will convert this issue and follow the discussion there.

@scikit-image scikit-image locked and limited conversation to collaborators Nov 6, 2021
@rfezzani rfezzani closed this as completed Nov 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
⏩ type: Enhancement Improve existing features
Projects
None yet
Development

No branches or pull requests

3 participants