<div dir="ltr"><div>Hi Mistral team,<br><br></div>We have the following critical errors in Mistral <a href="https://github.com/stackforge/mistral">repository</a>, whose I suggest to fix:<br><br>Code <a href="http://pylint-messages.wikidot.com/messages:e1001">E1101</a>, occurrences: 231<br>
Code <a href="http://pylint-messages.wikidot.com/messages:e1002">E1002</a>, occurrences: 9<div><div>Code <a href="http://pylint-messages.wikidot.com/messages:e1003">E1003</a>, occurrences: 2<br>Code <a href="http://pylint-messages.wikidot.com/messages:f0401">F0401</a>, occurrences: 21<br>
Code W0613, occurrences: 19 (Unused argument <em>%r</em>)<br>Code W0621, occurrences: 3  (<em>Redefining name %r from outer scope (line %s)</em>
Used when a variable’s name hide a name defined in the outer scope.)<br></div><div>Code <a href="http://pylint-messages.wikidot.com/messages:w0231">W0231</a>, occurrences: 3<br></div><div>Code <a href="http://pylint-messages.wikidot.com/messages:w0212">W0212</a>, occurrences: 3<br>
</div><div>Code W0612, occurrences: 1 (unused variable)<br></div><div>Code <a href="http://pylint-messages.wikidot.com/messages:c0323">C0323</a>, occurrences: 1<br></div><div><br></div><div>To find these errors I just run:<br>
pylint --rcfile=.pylintrc mistral<br></div><div>in Mistral repository. Probably we can find the other errors, when it will be executed with other parameters.<br><br></div><div>This issue was described in <a href="https://bugs.launchpad.net/mistral/+bug/1281164">this bug report</a>.<br>
</div><div>I suggest to fix these errors and after that add this pylint command to tox.ini file and check the same errors for all new commits.<br></div><div><br></div><div>Any comments and help with fixes are welcome )<br>
</div><div><br></div><div>Thanks!<br></div><div><br></div><div>-- <br><div dir="ltr"><font color="#888888"><font color="#888888"><br></font></font><div style="font-family:arial;font-size:small">Timur,</div><div style="font-family:arial;font-size:small">
QA Lead</div><div style="font-family:arial;font-size:small">OpenStack Murano Project</div><div style="font-family:arial;font-size:small">Mirantis Inc</div></div>
</div></div></div>