Closed
Bug 852422
Opened 9 years ago
Closed 9 years ago
Update documentation for ManifestDestiny
Categories
(Testing :: Mozbase, defect)
Testing
Mozbase
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: k0scist, Assigned: wlach)
Details
Attachments
(1 file)
2.10 KB,
patch
|
k0scist
:
review+
|
Details | Diff | Splinter Review |
I noted today that https://wiki.mozilla.org/Auto-tools/Projects/ManifestDestiny is obselete. We should look at this page and make sure that everything lives at https://mozbase.readthedocs.org/en/latest/manifestdestiny.html . https://mozbase.readthedocs.org/en/latest/manifestdestiny.html also has an obselete link to Projects/Manifest: https://mozbase.readthedocs.org/en/latest/manifestdestiny.html#developing-manifestdestiny ; this should be removed. In general the documentation should be audited for correctness and adequacy, but moving into a brave new manifest world at least these glaring flaws should be fixed.
Assignee | ||
Comment 1•9 years ago
|
||
Yup, sounds good. Are you planning to take care of this? I would also like to update the documentation to focus more on the API and how to use manifestdestiny in a practical context, but we can cover that some other time.
Reporter | ||
Comment 2•9 years ago
|
||
Probably eventually, but since my backlog grows faster than I can clear it, I have no particular estimate as to when. Yeah, the documentation ain't great.
Assignee | ||
Comment 3•9 years ago
|
||
(In reply to Jeff Hammel [:jhammel] from comment #2) > Probably eventually, but since my backlog grows faster than I can clear it, > I have no particular estimate as to when. > > Yeah, the documentation ain't great. I had a look at the page and I don't see any essential information that isn't covered by the current ManifestDestiny documentation. I think the only thing to do is remove references to that wiki page.
Assignee | ||
Comment 4•9 years ago
|
||
This mostly just removes references to the wiki page. I snuck in some more minor cleanup of the documentation.
Assignee: nobody → wlachance
Attachment #726773 -
Flags: review?(jhammel)
Reporter | ||
Comment 5•9 years ago
|
||
Comment on attachment 726773 [details] [diff] [review] Update manifestdestiny documentation Thanks, Will. Any "snuck in" improvements very much appreciated
Attachment #726773 -
Flags: review?(jhammel) → review+
Reporter | ||
Comment 6•9 years ago
|
||
Should we kill most of the wiki page after this lands? Again, I'd remove but I don't have that kind of power.
Assignee | ||
Comment 7•9 years ago
|
||
Pushed: https://github.com/mozilla/mozbase/commit/3707de502376996111d97e76e1a81597aea9f9ef
Assignee | ||
Comment 8•9 years ago
|
||
(In reply to Jeff Hammel [:jhammel] from comment #6) > Should we kill most of the wiki page after this lands? Again, I'd remove > but I don't have that kind of power. I thought this was a good idea, so I did it: https://wiki.mozilla.org/Auto-tools/Projects/ManifestDestiny
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 9•9 years ago
|
||
Thanks, Will!
You need to log in
before you can comment on or make changes to this bug.
Description
•