Merge lp:~cprov/core-image-watcher/lint into lp:core-image-watcher

[Celso Providelo]

Running pylint checks before merging.

Running pylint checks before merging.

Warnings, Recommendations and Style checks are disabled for now, we only care about Errors.

[Thomi Richards (thomir-deactivatedaccount)]

My vote is not to merge this. As shown in the diff, pylint doesn't understand some pretty straight-forward things, and forces us to either make our code less clear, or litter linter disabling comments everywhere.

If silly errors are landing in trunk, let's make sure we do a better job of reviewing MPs, writing tests, and/or manually testing our code (as appropriate to the situation).

review: Needs Fixing

[Francis Ginther (fginther)]

I think we should consider pylint as an option in the next sprint. As late as it is in the current sprint, I think it will end up being a distraction trying to get already existing code to comply and not produce it's intended benefits (which could be good benefits) and we don't have that much new code to write for this sprint.

But since this work is already done, I'm ok landing it and seeing how it works. If we want to invest more pylint in the future, we need to start somewhere to know if it's actually going to help.

review: Approve

Unmerged revisions

17. By Celso Providelo on 2015-04-08

Fixing pylint issue.

16. By Celso Providelo on 2015-04-08

Running pylint checks before merging.

=== modified file 'called-by-tarmac.py'
--- called-by-tarmac.py	2015-04-01 01:12:25 +0000
+++ called-by-tarmac.py	2015-04-08 01:19:55 +0000
@@ -94,6 +94,13 @@
                 'setup.py',
                 'flake8',
             ],
+            [
+                os.path.join(ve_dir, 'bin', 'python3'),
+                'setup.py',
+                'lint',
+                '--lint-disable=W,C,R',
+                '--lint-reports=no'
+            ],
         )
         for cmd in all_cmds:
             ret = _run_command(cmd)

=== modified file 'core_image_watcher/__init__.py'
--- core_image_watcher/__init__.py	2015-04-03 02:05:16 +0000
+++ core_image_watcher/__init__.py	2015-04-08 01:19:55 +0000
@@ -152,7 +152,7 @@
 
     logger.info('Started!', extra=logging_extra)
 
-    amqp_uris = config.get('amqp', 'uris').split()
+    amqp_uris = str(config.get('amqp', 'uris')).split()
     location = config.get('image', 'location')
     channel = config.get('image', 'channel')
     device = config.get('image', 'device')

=== modified file 'test_requirements.txt'
--- test_requirements.txt	2015-04-01 01:12:25 +0000
+++ test_requirements.txt	2015-04-08 01:19:55 +0000
@@ -1,2 +1,3 @@
+flake8==2.4.0
+setuptools-lint==0.3
 testtools==1.7.1
-flake8==2.4.0