Skip to content

brainfood / videojs-contrib-hls

Go to a project
Toggle navigation
Toggle navigation pinning
cfd8ae91 by David LaPalomento
Options

Pull and rebase simultaneously and use non-interactive amend 

Simplify the PR rebasing. Make sure the editor isn't invoked to amend the merge commit with the changelog update. Move tests to after the change has been fully merged.
1 parent 0bb6541d
Inline Side-by-side
Showing 1 changed file with 4 additions and 5 deletions
......@@ -342,16 +342,15 @@
{ "get": "{{meta.urls.repo_api}}/pulls/{{prNum}}", "desc": "Get the PR information", "id": "pr" },
{ "get": "{{meta.urls.repo_api}}/pulls/{{prNum}}/commits", "desc": "Get the PR commits to access author info", "id": "prCommits" },
[ "git checkout -b {{pr.user.login}}-{{pr.head.ref}} {{pr.base.ref}}", "Create a new branch for merging the changes" ],
[ "git pull {{pr.head.repo.ssh_url}} {{pr.head.ref}}", "Merge in the changes"],
[ "git rebase {{pr.base.ref}}", "Rebase against the latest changes in the base branch"],
[ "git pull --rebase {{pr.head.repo.ssh_url}} {{pr.head.ref}}", "Merge and rebase the changes"],
[ "git checkout {{pr.base.ref}}", "Check out the base branch" ],
{ "prompt": "confirm", "desc": "Does everything look ok?" },
{ "prompt": "text", "id": "line", "desc": "Describe this change in one line" },
[ "git merge --no-ff {{pr.user.login}}-{{pr.head.ref}} -m {{line}}. Closes #{{prNum}}", "Merge the changes" ],
[ "grunt test", "Run tests to make sure they still pass" ],
[ "grunt chg-add:'{{line}} ([view](https\\://github.com/{{meta.org}}/{{meta.name}}/pull/{{prNum}}))'", "Add a line to the changelog" ],
[ "git add CHANGELOG.md", "Add the changlelog change to be committed" ],
[ "git commit --amend", "Add the changelog change to the merge commit" ],
[ "git commit --amend --no-edit", "Add the changelog change to the merge commit" ],
[ "grunt test", "Run tests to make sure they still pass" ],
{ "prompt": "confirm", "desc": "Does everything look ok?" },
[ "git push origin {{pr.base.ref}}", "Push the changes to your remote copy of the project" ],
[ "git push upstream {{pr.base.ref}}", "Push the changes to the main project" ],
[ "git branch -D {{pr.user.login}}-{{pr.head.ref}}", "Delete the local branch used for merging" ]
......