-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Updated screenshot and information about changing base branch #1897
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
Conversation
…for pull requests
You could add some more detail about the branches. We get lots and lots of questions about that. You can tell them about the We should also update the |
Yes, I have/had these questions myself and while we're at it: Say I Am 05.11.2012 21:48, schrieb Wouter J:
::::::::::::::::::::::::::::::::::::::: Björn Fromme dreipunktnull Am Langen Siepen 18 Telefon: +49 (0) 2191 - 4 61 58 36 E-Mail: fromme@dreipunktnull.com ::::::::::::::::::::::::::::::::::::::: |
Great job! 👍 If the typo is inside this file you can put it here, otherwise I recommend to create a new PR. EDIT: I readed to fast.. For instance, if you create a PR for |
😊 Thanks. I created another PR. Am 06.11.2012 11:26, schrieb Wouter J:
::::::::::::::::::::::::::::::::::::::: Björn Fromme dreipunktnull Am Langen Siepen 18 Telefon: +49 (0) 2191 - 4 61 58 36 E-Mail: fromme@dreipunktnull.com ::::::::::::::::::::::::::::::::::::::: |
Updated screenshot and information about changing base branch
Hi again Björn! This is a very nice PR! I've merged it in with only a few tweaks. Thanks so much! |
Well, it pays to be a native speaker ;-) Thanks Edit: What about the image? I see the textual changes have been merged but the new screenshot is missing. |
No description provided.