837872 - test-manifest.ini references a 404 NOT FOUND location

Status: RESOLVED FIXED

(Testing :: Mozbase, defect)

Description

[Jeff Hammel]

https://github.com/mozilla/mozbase/blob/master/test-manifest.ini#L6
refers to
https://github.com/mozilla/mozbase/blob/master/manifestdestiny/README.txt
. This now no longer exists.  There is a resource,
https://github.com/mozilla/mozbase/blob/master/manifestdestiny/README.md
,  but it just points to
http://mozbase.readthedocs.org/en/latest/manifestdestiny.html . We
should probably just use this to link to.

Comment 1

[Jeff Hammel]

Attachment: proposed fix

Comment 2

[William Lachance (:wlach)]

Comment on attachment 710042 [details] [diff] [review]
proposed fix

I would personally just take out the link altogether as it's IMO not that useful and liable to get out of date again. But this is also fine.

Comment 3

[Jeff Hammel]

pushed: https://github.com/mozilla/mozbase/commit/7f200e66bb1639fbbc8db2bb2362740b46bb7a00

I tend towards leaving the URL in.  If we're committed to readthedocs as our long-term solution, then this URL should not change unless we significantly change our code structure or documentation strategy.  I've also heard a lot of complaints about the manifest format being hard to find the documentation for so I am inclined to keep this here.