Skip to content

Updates to script for 2-14-maintenance #55

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Feb 10, 2014
Merged

Updates to script for 2-14-maintenance #55

merged 3 commits into from
Feb 10, 2014

Conversation

JonRowe
Copy link
Member

@JonRowe JonRowe commented Feb 10, 2014

These are the updates I should have pushed earlier, sorry @myronmarston, I
basically extracted the "should we run support" stuff into a function so that
2-14 could be added to the list of builds not to run it on.

I also added to the Rakefile because partly aborted updates werent cleaning up
after themselves so this deletes the branch or renames the branch as desired.

elsif input =~ /^r(?:ename)? (.*)$/
branch_name = $1
end
end
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks pretty useful but the rename thing confused me a bit -- I assumed it was renaming the branch that was already there, rather than using a different name for the branch you are creating.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reworded.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note, this diff doesn't show that due to @github "feature", this diff is outdated it just doesn't admit it.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for me.

@myronmarston
Copy link
Member

LGTM

…h, also handles conflict with existing branches
JonRowe added a commit that referenced this pull request Feb 10, 2014
Updates to script for 2-14-maintenance
@JonRowe JonRowe merged commit 5e2e169 into master Feb 10, 2014
@JonRowe JonRowe deleted the 214_travis branch February 10, 2014 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants