11519 2017-05-17 14:46:26 +0000 check for four-space indented in python metadata 2017-11-30 20:46:17 +0000 1 1 6 Yocto Project Subprojects Patchwork/Patchtest Patchtest Testsuite 2.4 x86 Multiple RESOLVED OBSOLETE Medium enhancement 2.5 M2 1 leonardo.sandoval.gonzalez daniela.plascencia bluelightning ross.burton --- --- No (bug/feature does not impact docs) oldest_to_newest 73297 0 leonardo.sandoval.gonzalez 2017-05-17 14:46:26 +0000 According to [1] "Python functions must be four space indented - no tabs." [1] http://www.openembedded.org/wiki/Styleguide#Format_Guidelines 73326 1 daniela.plascencia 2017-05-18 15:04:23 +0000 (In reply to comment #0) > According to [1] "Python functions must be four space indented - no tabs." > > [1] http://www.openembedded.org/wiki/Styleguide#Format_Guidelines There's a Pylint option that catches tab indentation. We are already using Pylint to test Python code and I think changing a couple parameters will do the work. Do you think a new check is needed? 73327 2 leonardo.sandoval.gonzalez 2017-05-18 15:07:16 +0000 (In reply to comment #1) > (In reply to comment #0) > > According to [1] "Python functions must be four space indented - no tabs." > > > > [1] http://www.openembedded.org/wiki/Styleguide#Format_Guidelines > > There's a Pylint option that catches tab indentation. We are already using > Pylint to test Python code and I think changing a couple parameters will do > the work. Do you think a new check is needed? understood, then the pylint implementation should be the path to take. 77113 3 daniela.plascencia 2017-09-25 17:52:32 +0000 Changing to 2.5 M1. I'm planning to tackle most of the test-suite enhancements on such milestone. 77738 4 leonardo.sandoval.gonzalez 2017-10-20 14:45:23 +0000 This is not a M+, and can be work on M2. 78439 5 daniela.plascencia 2017-11-30 20:46:17 +0000 The test already checks for indentation errors/inconsistencies, but this issue is now obsolete as the pylint test case has to be refactored. I'm filing a new issue for tracking the refactoring process.