Skip to content

Issue #52: Add a --dry-run option - #53

Merged
ncoghlan merged 1 commit into
python:masterfrom
ncoghlan:issue-52-add-cherry-pick-dry-run-option
Mar 19, 2017
Merged

ncoghlan merged 1 commit into
python:masterfrom
ncoghlan:issue-52-add-cherry-pick-dry-run-option

Conversation

@ncoghlan

@ncoghlan ncoghlan commented Mar 18, 2017 •

Copy link
Copy Markdown
Contributor
  • prints git commands that would be executed if
    everything runs without errors
  • also prints the PR creation URLs that would be opened
  • does NOT do any local checkouts, cherry-picks, branch
    creation or branch deletion
  • does NOT actually create any PRs

Also includes a few associated changes related to getting
helpful dry-run output with my configuration:

  • prevents stderr output when checking for 'upstream' remote
  • new '--push' option to specify PR remote (I use
    origin/pr rather than upstream/origin for fetch/push)
  • adds a 'backport-' prefix to the PR branch names

Closes #52

- prints git commands that would be executed if
  everything runs without errors
- also prints the PR creation URLs that would be opened
- does NOT do any local checkouts, cherry-picks, branch
  creation or branch deletion
- does NOT actually create any PRs

Also includes a few associated changes related to getting
helpful dry-run output with my configuration:

* prevents stderr output when checking for 'upstream' remote
* new '--push' option to specify PR remote (I use
  origin/pr rather than upstream/origin for fetch/push)
* adds a 'backport-' prefix to the PR branch names
@ncoghlan ncoghlan changed the title Issue #52: Adds a new --dry-run option Issue #52: Add a --dry-run option Mar 18, 2017

@Mariatta Mariatta left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks great! Thanks! :)

@ncoghlan
ncoghlan merged commit 901fe2a into python:master Mar 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a --dry-run option to preview cherry-pick actions

3 participants