Merge lp://staging/~whosdaz/ubuntu-filemanager-app/gci6198315054006272 into lp://staging/ubuntu-filemanager-app

Proposed by Matthew Allen
Status: Superseded
Proposed branch: lp://staging/~whosdaz/ubuntu-filemanager-app/gci6198315054006272
Merge into: lp://staging/ubuntu-filemanager-app
Diff against target: 447 lines (+237/-175)
4 files modified
README (+14/-169)
README.autopilot (+55/-0)
README.developers (+154/-0)
README.translations (+14/-6)
To merge this branch: bzr merge lp://staging/~whosdaz/ubuntu-filemanager-app/gci6198315054006272
Reviewer Review Type Date Requested Status
Andrew Hayzen (community) Needs Fixing
Ubuntu File Manager Developers Pending
Review via email: mp+281280@code.staging.launchpad.net

This proposal has been superseded by a proposal from 2015-12-23.

To post a comment you must log in.
Revision history for this message
Andrew Hayzen (ahayzen) wrote :

This looks good so far, however for the other READMEs we have opted to use the markdown format, so to remain consistent with the other coreapps could this be updated? Also the web links should be in the format [title](url), you can use the weather mp [0] for reference. Sorry this should have been mentioned in the code-in task, but it hadn't been updated.

0 - https://code.launchpad.net/~emailgirishrawat/ubuntu-weather-app/markdown-readmes/+merge/280787

review: Needs Fixing
513. By Matthew Allen

Fixed Markdown Syntax

514. By Matthew Allen

Changed file extensions to .md

515. By Matthew Allen

Fixed minor syntax issues

Unmerged revisions

515. By Matthew Allen

Fixed minor syntax issues

514. By Matthew Allen

Changed file extensions to .md

513. By Matthew Allen

Fixed Markdown Syntax

512. By Matthew Allen

removed old readme text

511. By Matthew Allen

fixed readmes

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches