@skatsaounis I don't see why this needs to be added as part of our CI build. That's continuous integration, not continuous deployment. The result of running this command on the CI will be lost.

Different would be if this is added as a pre-commit hook so that anyone adding a new git commit would be forced to run this before that. If you manage to do that I'll happily review the PR again. That said, why have you ignored the following 2 issues? They seem correct:

• https://lintlyci.github.io/Flake8Rules/rules/E123.html
• https://lintlyci.github.io/Flake8Rules/rules/E125.html

Hi @lbalmaceda,

Thank you for looking into my PR.

I will look into pre commit hook but I have never done it before so a link would be useful to try that out.

As far as these two rules are concerned:

• E123: is invalid rule for PEP8. You can have a look at the bottom of the intentation where an example exists with my_list variable
• E125: After I change it to E129, because I just found that the erroneous case was separated, then it can be ignored. You can have a look here and here

We appreciate the effort here but I think this PR is going to age poorly as we merge in other PRs. Definitely open to discussing again at some point but going to close this for now.