-
Notifications
You must be signed in to change notification settings - Fork 7
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
Integrate pushtip with github's pr #10
Comments
Wishlist: "save" the coverletter/pull-request-description so you can start from it when writing a new one. |
A note to @cmarcelo's suggestion: save it even if there's an error on creating a pull request so that you don't need to write all again on a second attempt. I remember struggling with that sort of problem when using git-send-branch. |
@guludo git send-branch saves everything under .git/.send-branch |
@lucasdemarchi so it didn't reloaded when I tried again... |
@guludo maybe because git-send-branch is very stupid... it doesn't reload anything. Ii just leaves the patches there in But if you call it again it erases the previous one heheh. Until now nobody worried to fix it because once I explain what is happening people just memorize how to workaround it :-\ |
@lucasdemarchi hehe Ok, just created an issue regarding this question with git-send-branch. :) |
Maybe creating an issue to group PR versions would be a good idea? |
Another thing that might help is automatically tagging issues/PRs with |
I like the ideas. Are you going to implement this? |
I can give a try if nobody steps in. |
However -- this has to be well thought of. For instance, unless one can verify who added the |
@lpereira |
pushtip was originally conceived for working with gitolite. Now that we need to survive the github era, it would be nice to have it better integrated with github's pull request.
Part of this work is already being done by #9. What more would be nice?
@cmarcelo @guludo anything else you have in mind?
The text was updated successfully, but these errors were encountered: