Closed Bug 1056008 Opened 7 years ago Closed 7 years ago

Remove tbpl-manifest.ini


(Firefox OS Graveyard :: Gaia::UI Tests, defect)

Not set


(Not tracked)



(Reporter: zcampbell, Assigned: zcampbell)




(2 files)

After bug 1056007 is resolved then tbpl-manifest.ini will no longer be used.

It can be removed, or if that's a problem, its contents replaced with a note informing TBPL users where to find their test manifests now.
Bug 1056007 has not been resolved but the TBPL manifest has stopped being used.

I'll replace the tbpl-manifest.ini's contents with instructions with where to find the main manifest files.
Assignee: nobody → zcampbell
Attached file github pr
Expect the usual back and forth about the text content :)
Attachment #8506790 - Flags: feedback?(florin.strugariu)
Attachment #8506790 - Flags: feedback?(dave.hunt)
Comment on attachment 8506790 [details] [review]
github pr

I'd just remove the file and add the documentation to the README or (even better) MDN.
Attachment #8506790 - Flags: feedback?(dave.hunt) → feedback-
I was thinking long term we remove it, but short term I would like to make the transition to the new files as smooth as possible. I think that if I removed the file completely I'd get pinged about it, wasting my time.
Comment on attachment 8506790 [details] [review]
github pr

I agree with Dave on this one. 
But I understand Zac's point about getting pinged.

R+ if Zac promises to remove the file and add the final doc in the mdn or readme file
Attachment #8506790 - Flags: feedback?(florin.strugariu) → feedback+
Scout's honour, I will remove the file in 60 days :)

I'll work on the MDN docs now too (forgot about that).
I've updated the MDN docs to remove references to tbpl-manifest.ini

Dave I promise we'll delete the file in a few weeks.

Leave this open for someone to scoop up once we've thoroughly settled into the new manifest configuration.
QA Whiteboard: [fxosqa-auto-backlog-]
I think we can remove this file now
Attachment #8544553 - Flags: review?(dave.hunt)
Attachment #8544553 - Flags: review?(dave.hunt) → review+
merged in
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.