Merge lp:~canonical-platform-qa/autopilot/launch-cmd-doc into lp:autopilot
| Status: | Superseded |
|---|---|
| Proposed branch: | lp:~canonical-platform-qa/autopilot/launch-cmd-doc |
| Merge into: | lp:autopilot |
| Diff against target: |
43 lines (+28/-1) 1 file modified
docs/tutorial/running_ap.rst (+28/-1) |
| To merge this branch: | bzr merge lp:~canonical-platform-qa/autopilot/launch-cmd-doc |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| PS Jenkins bot | continuous-integration | Needs Fixing on 2015-01-19 | |
| Christopher Lee (community) | Needs Fixing on 2015-01-19 | ||
| Nicholas Skaggs (community) | Needs Fixing on 2015-01-16 | ||
| Allan LeSage (community) | 2015-01-15 | Needs Fixing on 2015-01-15 | |
|
Review via email:
|
|||
Commit Message
Add a new section that includes how to use autopilot launch command. Also update references of autopilot to autopilot3 in existing material.
Description of the Change
Add a new section that includes how to use autopilot launch command. Also update references of autopilot to autopilot3 in existing material.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:521
http://
Executed test runs:
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild:
http://
| Allan LeSage (allanlesage) wrote : | # |
Agree with Nick's suggestion concerning placement of launch doc on page, else looks fine.
- 522. By Richard Huddie on 2015-01-16
-
Fix review comments, move launch before vis, add -i option and also remove the 3s as this will be done in another mp
- 523. By Richard Huddie on 2015-01-16
-
remove last autopilot3 reference
| Richard Huddie (rhuddie) wrote : | # |
Thanks. I have made updates accordingly. I have also removed the changes to use autopilot3, so this will be easier to merge with other updates.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:522
http://
Executed test runs:
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild:
http://
| Nicholas Skaggs (nskaggs) wrote : | # |
Thanks for removing the autopilot3 refs; should be easier to merge multiple mp's now. Some more nit picky stuff, but I think we're close. Thanks Richard!
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:523
http://
Executed test runs:
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild:
http://
| Christopher Lee (veebers) wrote : | # |
There is currently text in the faq covering this (titled: 'Autopilot Qt & Gtk Support'). Perhaps we need to amalgamate these sections so we're not repeating ourselves. (Perhaps in the faq section was a silly place to put this initially :-)).
- 524. By Richard Huddie on 2015-01-19
-
Address latest review comments. Add app parameters, include framework error, use some info from faq page
| Richard Huddie (rhuddie) wrote : | # |
Thanks Nicholas, Chris, I have made updates to fix those issues.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:524
http://
Executed test runs:
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild:
http://
- 525. By Thomi Richards on 2015-01-20
-
Various cleanups.

Left comments about ordering and expansion.