Closed
Bug 1055223
Opened 12 years ago
Closed 12 years ago
Convert Testopia documentation to .rst format
Categories
(Testopia :: General, enhancement)
Tracking
(Not tracked)
RESOLVED
FIXED
3.0
People
(Reporter: u430950, Assigned: u430950)
Details
Attachments
(1 file, 1 obsolete file)
|
81.54 KB,
patch
|
LpSolit
:
review+
|
Details | Diff | Splinter Review |
In order to take advantage of the new documentation format, as well as improve the quality of the Testopia documentation itself by making it easier to work with, Testopia should include an .rst version of its docs.
Assignee: gregaryh → theycallmefish
Status: NEW → ASSIGNED
Attachment #8474747 -
Flags: review?(LpSolit)
Comment 2•12 years ago
|
||
Comment on attachment 8474747 [details] [diff] [review]
testopia.diff
The doc compiles correctly. The URL to download Bugzilla doesn't work, but it can be fixed on checkin (typo in the URL). Requirements mention JSON 1.14, but I have JSON 2.59 on my machine and Testopia is working correctly, so I assume the recommended version could be bumped, or even left blank. Feel free to file a separate bug if needed. r=LpSolit
Do you still want the other Manual.* files in the repo, or should they be removed now that you have this RST file?
Attachment #8474747 -
Flags: review?(LpSolit) → review+
Updated•12 years ago
|
Target Milestone: --- → 2.6
Comment 3•12 years ago
|
||
One more thing about your patch: lines are way too long and should be split (at around 80 characters per line), for readability. Do you want to submit a new patch to fix that?
I will submit a new patch with the changes you have suggested in comment 2 and comment 3. The document content is, for the most part, a copy/paste of the PDF file, with glaring mistakes corrected and proper formatting. I'm hoping to fix up some of the more lacking documentation areas (such as test environments) while working on bug 812482.
I would honestly keep the existing manual files in the system until Bugzilla 5.0 is the oldest version still being supported, as it will be the first version that would be able to utilize this new RST format.
Attachment #8474747 -
Attachment is obsolete: true
Attachment #8475243 -
Flags: review?(LpSolit)
Comment 6•12 years ago
|
||
Comment on attachment 8475243 [details] [diff] [review]
testopiav2.diff
r=LpSolit
Attachment #8475243 -
Flags: review?(LpSolit) → review+
Comment 7•12 years ago
|
||
I cannot remember if you have commit privs to the git repos or not... Tell me if you want me to commit your patch.
I do not have commit privileges to the git repos, so, at least for the time being, any changes made by me will need to be pushed to git by someone else.
Comment 9•12 years ago
|
||
To ssh://gitolite3@git.mozilla.org/bugzilla/extensions/testopia.git
567f865..f674204 master -> master
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•