Closed Bug 857535 Opened 11 years ago Closed 11 years ago

Remove the tree hasGroups property, since it is no longer needed

Categories

(Tree Management Graveyard :: TBPL, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: emorley)

References

Details

Attachments

(1 file)

* 'hasGroups' was added in bug 525377 since the Firefox3.6 tree didn't have chunked mochitests.
* The vast majority of trees in Config.js now have 'hasGroups' set to true.
* For those that don't, nothing would break for them even if it was, since they don't match against the grouped testsuites list in Config.js (helpfully called groupedMachineTypes).

As such, might as well remove 'hasGroups' to shorten the repo configs in Config.js
Attached patch Patch v1Splinter Review
Attachment #732769 - Flags: review?(ryanvm)
Attachment #732769 - Flags: review?(ryanvm) → review+
Depends on: 857996
In production :-)
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Product: Webtools → Tree Management
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: