Merge lp:~dholbach/help-app/1430326 into lp:~ubuntu-touch-coreapps-drivers/help-app/trunk

Proposed by Daniel Holbach
Status: Merged
Merged at revision: 84
Proposed branch: lp:~dholbach/help-app/1430326
Merge into: lp:~ubuntu-touch-coreapps-drivers/help-app/trunk
Diff against target: 13 lines (+2/-1)
1 file modified
edit-here/ (+2/-1)
To merge this branch: bzr merge lp:~dholbach/help-app/1430326
Reviewer Review Type Date Requested Status
David Planella Approve
Review via email:
To post a comment you must log in.
Revision history for this message
David Planella (dpm) wrote :

LGTM. Just one suggestion - perhaps testing for MIME type (text/plain or text/x-markdown) might be more robust?

Either way should work.

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

I'll file a separate bug for that.

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'edit-here/'
2--- edit-here/ 2015-03-05 09:32:33 +0000
3+++ edit-here/ 2015-03-10 13:43:48 +0000
4@@ -213,7 +213,8 @@
5 def __init__(self):
6 = []
7 for dirpath, dirnames, fns in os.walk(PATH):
8- for fn in fns:
9+ for fn in [fn for fn in fns
10+ if fn.endswith('.md')]:
11 fn = os.path.relpath(os.path.join(dirpath, fn),
12 os.path.join(PATH, '..'))
13 += [fn]


People subscribed via source and target branches