Merge lp://qastaging/~elopio/selenium-simple-test/fix1170389-actions_docstrings into lp://qastaging/selenium-simple-test

Proposed by Leo Arias
Status: Merged
Approved by: Corey Goldberg
Approved revision: 415
Merged at revision: 412
Proposed branch: lp://qastaging/~elopio/selenium-simple-test/fix1170389-actions_docstrings
Merge into: lp://qastaging/selenium-simple-test
Diff against target: 1568 lines (+617/-339)
2 files modified
docs/index.rst (+4/-4)
src/sst/actions.py (+613/-335)
To merge this branch: bzr merge lp://qastaging/~elopio/selenium-simple-test/fix1170389-actions_docstrings
Reviewer Review Type Date Requested Status
Corey Goldberg (community) Approve
Review via email: mp+165697@code.qastaging.launchpad.net

Commit message

Updated the actions docstrings to follow pep257 suggestions with the sphinx markup.

To post a comment you must log in.
412. By Leo Arias

Typo.

Revision history for this message
Corey Goldberg (coreygoldberg) wrote :

argument is misspelled as "argumetn" in a few places.

besides that, this is great!
I ran the docs through sphinx.. here is the latest:
https://chinstrap.canonical.com/~cgoldberg/sst_docs/actions.html

index.rst should also be updated with the new wording you used:
"Tests are comprised of Python scripts or test case classes."

413. By Leo Arias

Fix argumetn typos.

414. By Leo Arias

Fix arguments typos.

415. By Leo Arias

Updated the index to mention classes.

Revision history for this message
Corey Goldberg (coreygoldberg) wrote :

awesome.
+1

review: Approve

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