diff options
author | David James <davidjames@google.com> | 2013-09-18 17:57:13 -0700 |
---|---|---|
committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2013-09-30 18:58:23 +0000 |
commit | c7c4ff5f338b602e67bbe9ce5bb0cfeb8e499bfd (patch) | |
tree | a9928dd11bffc8a97b1af2e2b72baff27bc39697 /scripts/cros_mark_as_stable.py | |
parent | d810e51da2145145247d52e7284afdf82dac74ac (diff) | |
download | chromite-c7c4ff5f338b602e67bbe9ce5bb0cfeb8e499bfd.tar.gz |
Always clean up created branches.
On trybots, we currently skip some of the steps for cleaning up git
repositories. Don't do that.
Relatedly, also ensure that branches in a git repo are cleaned up even
if the project isn't checked out, and add a docstring to the class in
cros_mark_as_stable that uses one of these branches.
BUG=chromium:293518
TEST=Verify new command blows away branches.
TEST=Unit tests.
TEST=mario-paladin trybot run.
Change-Id: Ib48746430924c7b9a5dba3ddd2a95f1ec6277d84
Reviewed-on: https://chromium-review.googlesource.com/169925
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: David James <davidjames@chromium.org>
Tested-by: David James <davidjames@chromium.org>
Diffstat (limited to 'scripts/cros_mark_as_stable.py')
-rw-r--r-- | scripts/cros_mark_as_stable.py | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/scripts/cros_mark_as_stable.py b/scripts/cros_mark_as_stable.py index 898abdd79..577de227c 100644 --- a/scripts/cros_mark_as_stable.py +++ b/scripts/cros_mark_as_stable.py @@ -162,7 +162,13 @@ class GitBranch(object): """Wrapper class for a git branch.""" def __init__(self, branch_name, tracking_branch, cwd): - """Sets up variables but does not create the branch.""" + """Sets up variables but does not create the branch. + + Arguments: + branch_name: The name of the branch. + tracking_branch: The associated tracking branch. + cwd: The git repository to work in. + """ self.branch_name = branch_name self.tracking_branch = tracking_branch self.cwd = cwd |