Closed
Bug 1083521
Opened 11 years ago
Closed 11 years ago
Remove old CI
Categories
(Firefox OS Graveyard :: Gaia, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
2.1 S7 (24Oct)
People
(Reporter: kgrandon, Assigned: kgrandon)
Details
(Whiteboard: [systemsfe])
Attachments
(3 files)
No description provided.
|
Assignee | |
Comment 1•11 years ago
|
||
|
|
Assignee | |
Comment 2•11 years ago
|
||
Attachment #8505791 -
Flags: review?(jlal)
|
|
Assignee | |
Updated•11 years ago
|
Attachment #8505788 -
Flags: review?(jlal)
|
|
Assignee | |
Comment 3•11 years ago
|
||
Comment on attachment 8505791 [details] [review]
Pull request - Remove travis-project-jobs
Gareth - maybe you could help me review these?
Attachment #8505791 -
Flags: review?(gaye)
|
|
Assignee | |
Updated•11 years ago
|
Attachment #8505788 -
Flags: review?(gaye)
|
||
Comment 4•11 years ago
|
||
So if we delete this stuff, then prs look fine, but would it be at all reasonable to keep that stuff around in case we decide to do travis again someday?
|
|
Assignee | |
Comment 5•11 years ago
|
||
(In reply to Gareth Aye [:gaye] from comment #4)
> So if we delete this stuff, then prs look fine, but would it be at all
> reasonable to keep that stuff around in case we decide to do travis again
> someday?
Travis is perma-red for almost all jobs, so I don't think it makes sense to keep this stuff in the codebase, especially when it could impact grepability. I think we should remove it, and we can always roll back the files, or grab them from the 2.0 branch - thanks to source control.
Basically Travis is broken, and not easily fixed. I don't see any benefit to keep these files around, especially since TC already surpasses Travis today.
|
|
||
Updated•11 years ago
|
Attachment #8505788 -
Flags: review?(gaye) → review+
|
|
||
Updated•11 years ago
|
Attachment #8505791 -
Flags: review?(gaye) → review+
|
|
||
Comment 6•11 years ago
|
||
Ok sounds good to me... thanks!
|
|
Assignee | |
Comment 7•11 years ago
|
||
Comment on attachment 8505788 [details] [review]
Pull request - Remove old CI scripts from gaia
Thanks Gareth!
Attachment #8505788 -
Flags: review?(jlal)
|
|
Assignee | |
Updated•11 years ago
|
Attachment #8505791 -
Flags: review?(jlal)
|
|
Assignee | |
Comment 8•11 years ago
|
||
Travis scripts removed from gaia: https://github.com/mozilla-b2g/gaia/commit/c3249ea26d29a6a61530301d5d176305361dcd39
Removed from gaia-node-modules: https://github.com/mozilla-b2g/gaia-node-modules/commit/f3ddc6a219123208cb1a1b2a17e5671609f3e025
|
|
Assignee | |
Comment 9•11 years ago
|
||
|
|
Assignee | |
Comment 10•11 years ago
|
||
Node modules bump in master: https://github.com/mozilla-b2g/gaia/commit/6aa47bae89b9930527fd06b3eaeeb052757cb3d6
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
|
||
Updated•11 years ago
|
Whiteboard: [systemsfe]
Target Milestone: --- → 2.1 S7 (24Oct)
You need to log in
before you can comment on or make changes to this bug.
Description
•