-
Notifications
You must be signed in to change notification settings - Fork 660
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
Learn to write good commit message and description #1891
Comments
There's also a number of things I encourage people to learn about Git. Maybe you will find something for yourselves.
Extras: |
A serious resource! I wonder should you have a canonical URL on http://webknjaz.me/ :) |
@decentral1se what do you mean about canonical URL there? :) Are you suggesting to put up a web page with those links? I normally post this as GitHub issues/tasks in repos for my mentees... |
Fixes master by performing fixes that were not tested by Travis due to current config. Signed-off-by: Sorin Sbarnea <ssbarnea@redhat.com>
Read a good advice recently: https://twitter.com/jefflembeck/status/1118892291995734016 |
While this ticket is still very useful I will be closing it because there is no end to it as it not molecule specific. Most of this coud be included in a docs page but the next question would be: why to document this in molecule when we could just defer this to ansible itself. |
It's a follow-up for:
Originally posted by @webknjaz in #1883 (comment)
According to common commit style guides, it is required to write messages in an imperative manner. Make them actionable and atomic. Also avoid using
-m
as you often need to express more detail in a long way. Here's several articles sharing best practices:N.B. This all applies to issues and PRs as well. If you submit a patch, everything in diff should refer to only one logical change making it atomic. Ideally, applying a single PR should transition a project from one working state to another completely working state, meaning there shouldn't be a change submitted via two PRs and there shouldn't be anything more than needed to fulfill what's in description and title of the PR.
P.S. Don't scare (other) maintainers with huge changes. Reviewing a PR is hard work. The bigger it is the more chance is that somebody will postpone it until better times (trying to find an appropriate slot in their calendars to the review in one pass).
Smaller changes attract more viewers and people, in general, are more likely to understand tiny patches and their impact.
The text was updated successfully, but these errors were encountered: