-
Notifications
You must be signed in to change notification settings - Fork 177
Getting_In_Touch: Add information about long issue discussions #88
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
Help/Getting_In_Touch.rst
Outdated
| a bug, just file `an issue <https://github.com/coala-analyzer/coala/issues>`__ | ||
| and we will check it out! | ||
| and we will check it out. In case of long discussion, you should create | ||
| new issue with clearly defined task! |
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.
shouldn't it be "a new issue"?
|
Hi, it seems that your commit message doesn't contain enough information. Don't worry, most people don't know about good commit messages when they join us, including very experienced developers :) Standards vary from project to project. The most important part is the first line of your commit message - maintainers will use it to read through a large number of commits and thus it's important that it's short but most importantly concise. I'd heartily recommend reading http://coala.readthedocs.io/en/latest/Getting_Involved/Writing_Good_Commits.html ! To change your message you just use Cheers! |
|
Hey, I think you'll want to use Closes instead of Fixes - I'll talk about that in a few minutes in the workshop :) |
|
you'll want to reference the issue, not the PR! |
|
ack 8a97fbd |
|
@rultor merge |
@ladaradon @sils1297 Oops, I failed. You can see the full log here (spent 12s) |
|
@rultor merge |
@AsnelChristian OK, I'll try to merge now. You can check the progress of the merge here |
@AsnelChristian @ladaradon Oops, I failed. You can see the full log here (spent 11s) |
|
check #91 :) |
Closes #45