Skip to content
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

Added note about updating pull requests by force push #17026

Merged

Conversation

prathamesh-sonpatki
Copy link
Member

  • Added note about force pushing same branch to update a existing pull
    request instead of opening new one in contributing guide.
  • [ci skip]

- Added note about force pushing same branch to update a existing pull
  request instead of opening new one in contributing guide.
- [ci skip]
@prathamesh-sonpatki
Copy link
Member Author

Related - #16965 (comment)
This can be seen many times. The contributing guide has a note about squashing but does not say about updating a pull request.

@schneems
Copy link
Member

Thanks! Also good seeing you in Tokyo!

schneems added a commit that referenced this pull request Sep 23, 2014
…ull-requests

Added note about updating pull requests by force push
@schneems schneems merged commit 13e1348 into rails:master Sep 23, 2014
@prathamesh-sonpatki prathamesh-sonpatki deleted the mention-updating-pull-requests branch September 23, 2014 21:21
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.

None yet

2 participants