-
Notifications
You must be signed in to change notification settings - Fork 16
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
Add support stdin for hook #13
Conversation
Demonstrate the effect with a pre-push example: > git push
args: [
"origin",
"git@github.com:binsee/sidecar.git",
"refs/heads/test2",
"65b8bd5bd8e8c889fb8ec6228cfada35164cc423",
"refs/heads/test2",
"0000000000000000000000000000000000000000"
]
> git push origin
args: [
"origin",
"git@github.com:binsee/sidecar.git",
"refs/heads/test2",
"65b8bd5bd8e8c889fb8ec6228cfada35164cc423",
"refs/heads/test2",
"0000000000000000000000000000000000000000"
]
> git push origin test2
args: [
"origin",
"git@github.com:binsee/sidecar.git",
"refs/heads/test2",
"65b8bd5bd8e8c889fb8ec6228cfada35164cc423",
"refs/heads/test2",
"0000000000000000000000000000000000000000"
]
> git push origin :test
args: [
"origin",
"git@github.com:binsee/sidecar.git",
"(delete)",
"0000000000000000000000000000000000000000",
"refs/heads/test",
"65b8bd5bd8e8c889fb8ec6228cfada35164cc423"
]
> git push origin test2 test3
args: [
"origin",
"git@github.com:binsee/sidecar.git",
"refs/heads/test2",
"65b8bd5bd8e8c889fb8ec6228cfada35164cc423",
"refs/heads/test2",
"0000000000000000000000000000000000000000"
]
Total 0 (delta 0), reused 0 (delta 0)
To github.com:binsee/sidecar.git
* [new branch] test2 -> test2
* [new branch] test3 -> test3 |
Fixes Chatie#19 Need update git-scripts to new version link git-script/pr: zixia/git-scripts#13
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work, thank you very much for the improvement!
Could you please bump the minor version to an odd version, so that we can merge this PR and publish a new version on NPM?
Appreciate it!
done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, thank you very much for this PR!
* Re-push produces meaningless version when push fails Fixes #19 Need update git-scripts to new version link git-script/pr: zixia/git-scripts#13 * style: * ci: fix smoke-test * 0.6.3 * refactor: Comment out unused code * 0.6.4 * test: fix test error * 0.6.5 * ci: 💚 update nodejs version * 0.6.6 * fix: fix error on old version for git-scripts * 0.6.7 * fix: 🐛 fix npm bump again * 0.6.8 * build: ⬆️ update git-scripts to 0.5.3 * 0.6.9 * feat: ✨ suppport multi branch in pre-push * 0.6.10 * fix: * 0.6.11 * fix: 💚 fix error in test:pack * 0.6.12 * chore: ⬆️ upgrade git-scripts to 0.6.0 * 0.7.0 * fix: 🐛 fix read properties error * 0.7.1
Fixes #12