Markus Elfring
Markus Elfring
>… You just have to install MaxDB, … Do you find the proposed adjustment of the function “MaxdbConnection.sqlAddLimit()” reasonable?
>Too condensed. :thought_balloon: Do we present different views for [the usage of conditional expressions](https://docs.python.org/3/reference/expressions.html#conditional-expressions "Documentation") (in combination with “augmented” assignments)? (Another bit of duplicate source code specifications can eventually be...
By the way: :thought_balloon: Would there be a need to adjust [the check for the usage of a key word like “`WHERE`”](https://stackoverflow.com/questions/5319922/python-check-if-word-is-in-a-string "Check if word is in a string") any...
>Currently no, but I cannot give any promise for the future. Is the discussed check approach generally questionable? :thinking:
>Why do you care? :thought_balloon: It seems that we stumble on different coding style preferences.
>For some reasons you decided to go long way. There are special dependencies involved so that this patch review evolved somehow. * The status for the software component “MaxDB” might...
>How much time do you need to prepare new code, amend commit messages and force push? I would eventually prefer to offer another development branch. >Is a week enough? Software...
I suggest to avoid [ignorance of return values](https://cwe.mitre.org/data/definitions/252.html) a bit more. How do you think about to improve [static source code analysis](https://en.wikipedia.org/wiki/List_of_tools_for_static_code_analysis#C.2C_C.2B.2B) also for this software?
How do you think about the generation of update candidates like the following? :thinking: ``` Markus_Elfring@Sonne:/home/altes_Heim3/Markus_Elfring/Projekte/python-holidays/lokal> perl -p -i.orig -0777 -e 's/^(?\s+)(?\S+)\s*=\s*\k[ \t]*(?[+\-%&|^@]|\*\*?|\/\/?|)/$+{indentation}$+{target} $+{operator}=/gm' $(find . -name '*.py') && git...
Which Perl versions would hinder adjustments (which can generally be generated by the mentioned source code search and replacement approach)? :thought_balloon: Would any other contributor be quicker with the integration...