canmove, Confirmed users
1,126
edits
Tree Rules/Integration: Difference between revisions
Jump to navigation
Jump to search
m
→What are the tree rules for integration repositories?: Link to treeherder.m.o
(replace 'tinderbox' with 'treeherder') |
m (→What are the tree rules for integration repositories?: Link to treeherder.m.o) |
||
| Line 11: | Line 11: | ||
* Follow the '''[https://developer.mozilla.org/En/Developer_Guide/Committing_Rules_and_Responsibilities general rules for committing]''', ''except'': | * Follow the '''[https://developer.mozilla.org/En/Developer_Guide/Committing_Rules_and_Responsibilities general rules for committing]''', ''except'': | ||
** Committers are ''not'' required to actively watch the tree after pushing to an integration repository. | ** Committers are ''not'' required to actively watch the tree after pushing to an integration repository. | ||
** Checking treeherder before pushing is not required, but it is appreciated. If the tree is in a very broken state, you can save the sheriffs work by notifying them instead of pushing. | ** Checking [https://treeherder.mozilla.org/ Treeherder] before pushing is not required, but it is appreciated. If the tree is in a very broken state, you can save the sheriffs work by notifying them instead of pushing. | ||
* Integration repositories are no replacement for '''[[Try]]'''. You still need to test your patches before pushing. (This doesn't mean that you need an all-platforms try run for every patch. But it does mean that you should do enough testing so that you rarely cause red or orange on the integration repository. But breaking it rarely is ok. Never missing a plane means you're spending too much time in airports; never breaking the tree means you're running too many tests before landing.) | * Integration repositories are no replacement for '''[[Try]]'''. You still need to test your patches before pushing. (This doesn't mean that you need an all-platforms try run for every patch. But it does mean that you should do enough testing so that you rarely cause red or orange on the integration repository. But breaking it rarely is ok. Never missing a plane means you're spending too much time in airports; never breaking the tree means you're running too many tests before landing.) | ||
* When can I push to the tree? '''Always!''' | * When can I push to the tree? '''Always!''' | ||