HomePhabricator
Diffusion Scap 0cfc11b5783d

git.update_submodules DWIM fixes

Description

git.update_submodules DWIM fixes

Tags: Release-Engineering-Team

Summary: Allow using without having to pass a bogus remote we don't even need if we're going to use upstream. Raise a ValueError if you mismatch them though to keep people from shooting themselves in the foot.

Test Plan: Looked at it super duper closely

Reviewers: mmodell, Release-Engineering-Team, thcipriani

Reviewed By: Release-Engineering-Team, thcipriani

Subscribers: thcipriani, jenkins

Differential Revision: https://phabricator.wikimedia.org/D422

Details

Provenance
demonAuthored on Oct 25 2016, 4:54 PM
Reviewer
Release-Engineering-Team
Differential Revision
D422: git.update_submodules DWIM fixes
Parents
rMSCAaa8c568c9b8f: Fix service restart
Branches
Unknown
Tags
Unknown
ChangeId
None
Build Status
Buildable 1170
Build 1729: ci-jessieJenkins