Merge lp://staging/~kyrofa/unity-scopes-api/1452423_fix-tutorial-typos into lp://staging/unity-scopes-api

Proposed by Kyle Fazzari
Status: Superseded
Proposed branch: lp://staging/~kyrofa/unity-scopes-api/1452423_fix-tutorial-typos
Merge into: lp://staging/unity-scopes-api
Diff against target: 30 lines (+3/-3)
1 file modified
doc/tutorial.dox (+3/-3)
To merge this branch: bzr merge lp://staging/~kyrofa/unity-scopes-api/1452423_fix-tutorial-typos
Reviewer Review Type Date Requested Status
Michi Henning (community) Needs Fixing
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+258427@code.staging.launchpad.net

This proposal has been superseded by a proposal from 2015-05-07.

Commit message

Fix a couple typos in the tutorial.

Description of the change

This simply fixes a couple of typos found in the tutorial.

To test:

1) Look at old documentation, and notice the following sentence: "Prior to sending any queries, the scopes run time calls run() if your stop() method completed successfully (did not throw an exception)."

2) Regenerate documentation based on this MP:

$ cmake /path/to/src
$ make doc

3) Open up doc/index.html and notice that the sentence is now: "Prior to sending any queries, the scopes run time calls run() if your start() method completed successfully (did not throw an exception)." A few other smaller typos were also corrected.

To post a comment you must log in.
Revision history for this message
Michi Henning (michihenning) wrote :

Great, thanks for that!

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Michi Henning (michihenning) wrote :

I just realized that you targeted this at trunk, but it really should be devel. Nothing wrong with the MR as such, just the wrong target branch. Can you re-submit for the correct branch please?

review: Needs Fixing

Unmerged revisions

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

to all changes: