Merge lp://qastaging/~unit193/xubuntu-docs/cli into lp://qastaging/xubuntu-docs/trusty

Proposed by Unit 193
Status: Merged
Merged at revision: 137
Proposed branch: lp://qastaging/~unit193/xubuntu-docs/cli
Merge into: lp://qastaging/xubuntu-docs/trusty
Diff against target: 46 lines (+20/-5)
2 files modified
debian/changelog (+7/-0)
desktop-guide/C/command-line.xml (+13/-5)
To merge this branch: bzr merge lp://qastaging/~unit193/xubuntu-docs/cli
Reviewer Review Type Date Requested Status
Jack Fromm (community) Needs Fixing
Pasi Lallinaho (community) Approve
Review via email: mp+206317@code.qastaging.launchpad.net

Description of the change

Commit message said it all, just added more commands to the "Common Commands" section and re-ordered the list.

To post a comment you must log in.
Revision history for this message
Pasi Lallinaho (knome) :
review: Approve
Revision history for this message
Jack Fromm (jjfrv8) wrote :

I think the apt-cache, apt-get and wget entries need periods instead of commas to make new sentences at (...repository. Common...), (...in apt. Common...), (...locally. See man wget...) respectively.

Just my opinion, but I think it would be better to be consistent with the verbs; e.g., less says "View" but ls says "Lists". Either make them all 3rd person singular or all imperative (or infinitive, or whatever the heck that is).

In killall, "programs" should be "program's".

review: Needs Fixing

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