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

Proposed by Daniel Holbach
Status: Merged
Approved by: Daniel Holbach
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
Nicholas Skaggs (community) Approve
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.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Daniel Holbach (dholbach) wrote :

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

Revision history for this message
Nicholas Skaggs (nskaggs) :
review: Approve
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
123. By Daniel Holbach

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

Approved by Nicholas Skaggs, Ubuntu Phone Apps Jenkins Bot.

Revision history for this message
Daniel Holbach (dholbach) wrote :

Approving again, the necessary requirement has landed now.

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) :
review: Approve (continuous-integration)
124. By Daniel Holbach

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