Merge lp:~dholbach/help-app/run-pep8-and-pyflakes3 into lp:help-app

Proposed by Daniel Holbach on 2015-03-25
Status: Merged
Approved by: Daniel Holbach on 2015-03-25
Approved revision: no longer in the source branch.
Merged at revision: 124
Proposed branch: lp:~dholbach/help-app/run-pep8-and-pyflakes3
Merge into: lp:help-app
Diff against target: 47 lines (+8/-1)
4 files modified
HACKING (+1/-1)
Makefile (+1/-0)
debian/control (+2/-0)
internals/run-other-tests (+4/-0)
To merge this branch: bzr merge lp:~dholbach/help-app/run-pep8-and-pyflakes3
Reviewer Review Type Date Requested Status
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve on 2015-03-25
Nicholas Skaggs 2015-03-25 Approve on 2015-03-25
Review via email: mp+254059@code.launchpad.net

Commit Message

Run pyflakes3 and pep8 as part of the 'check' target.

To post a comment you must log in.
review: Needs Fixing (continuous-integration)
Daniel Holbach (dholbach) wrote :

Should pass tests once lp:~dholbach/help-app/1434415 is landed.

Nicholas Skaggs (nskaggs) :
review: Approve
review: Needs Fixing (continuous-integration)
123. By Daniel Holbach on 2015-03-25

Fix utopic build. Fixes: https://bugs.launchpad.net/bugs/1434415.

Approved by Nicholas Skaggs, Ubuntu Phone Apps Jenkins Bot.

Daniel Holbach (dholbach) wrote :

Approving again, the necessary requirement has landed now.

review: Approve (continuous-integration)
124. By Daniel Holbach on 2015-03-25

Run pyflakes3 and pep8 as part of the 'check' target.

Approved by Ubuntu Phone Apps Jenkins Bot, Nicholas Skaggs.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'HACKING'
2--- HACKING 2015-03-20 14:18:33 +0000
3+++ HACKING 2015-03-25 10:14:58 +0000
4@@ -86,7 +86,7 @@
5
6 sudo apt install python-pelican po4a make bzrtools \
7 ubuntu-html5-ui-toolkit python3-polib python3-magic \
8- python3-markdown
9+ python3-markdown pep8 pyflakes
10
11 This will install the necessary files, so you can build the app or web build
12 locally and check if your changes actually make sense and look and work well.
13
14=== modified file 'Makefile'
15--- Makefile 2015-03-20 12:25:41 +0000
16+++ Makefile 2015-03-25 10:14:58 +0000
17@@ -54,6 +54,7 @@
18
19 check:
20 cd $(INTERNALS_DIR); ./run-tests
21+ cd $(INTERNALS_DIR); ./run-other-tests
22
23 translations:
24 cd $(INTERNALS_DIR); ./generate-translations
25
26=== modified file 'debian/control'
27--- debian/control 2015-03-11 11:57:50 +0000
28+++ debian/control 2015-03-25 10:14:58 +0000
29@@ -4,7 +4,9 @@
30 Maintainer: Daniel Holbach <daniel.holbach@ubuntu.com>
31 Build-Depends: bzr,
32 debhelper (>= 9),
33+ pep8,
34 po4a,
35+ pyflakes,
36 python-pelican (>= 3.5.0~),
37 python3-magic,
38 python3-markdown,
39
40=== added file 'internals/run-other-tests'
41--- internals/run-other-tests 1970-01-01 00:00:00 +0000
42+++ internals/run-other-tests 2015-03-25 10:14:58 +0000
43@@ -0,0 +1,4 @@
44+#!/bin/sh -e
45+
46+pep8 --repeat --show-source .
47+pyflakes3 .

Subscribers

People subscribed via source and target branches