You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Some of these fit well, reads well, and potentially reduces the message size.
Here is an example list of what I try to follow. It also contains my modification to what I seen in the website above. Also, as a side note, I sometimes even forget or use or use the wrong tag:
feat - new feature
fix - bug fix
docs - documentation
This one may not be necessary in cordova-docs as almost 90% of changes should be doc related, but if we include blog for the blog posts, then we could continue to use docs for separation.
style - formatting
refactor - refactoring of code, variable renaming, etc
test - all test related changes (adding, refactoring, deleting)
I am not trying to resolve, change, or enforced some guideline immediately so please do not try push this through ASAP.
At of the time of this posting, I am heavily focused on preparing major releases and hoping to get them out. If someone didn't follow the current or proposed guideline, I do not see any harm has it has not been very strict in the recent years. It current goal is to not block contribution or the major release process. The merger also has a chance to change the commit message when squash & merge.
I don't want to take too much time or effort away form the major release right now.
The text was updated successfully, but these errors were encountered:
@brodybits here is the ticket about what I been doing for git commit messages. Again as I said in the ticket, I am only posting it right now for informational purpose and including others.. I am not seeking for immediate change or enforcement. I want to focus on the major releases right now.
Feature Request
Motivation Behind Feature
Improve the CHANGELOG readability by including to the git commit messages
type
andscope
.Feature Description
Change and or improve on our existing git commit message standards.
Background
Currently I been using a modified version of:
Some of these fit well, reads well, and potentially reduces the message size.
Here is an example list of what I try to follow. It also contains my modification to what I seen in the website above. Also, as a side note, I sometimes even forget or use or use the wrong tag:
feat
- new featurefix
- bug fixdocs
- documentationcordova-docs
as almost 90% of changes should be doc related, but if we includeblog
for the blog posts, then we could continue to usedocs
for separation.style
- formattingrefactor
- refactoring of code, variable renaming, etctest
- all test related changes (adding, refactoring, deleting)chore
- internal tasks (updating dependencies, etc)breaking
- to reference a breaking changeci
- CI related changes (update, add, delete CI recipes, services, etc.)Example usage:
We can also include
scoping
to increase understanding of where a change might have been and maybe even reduce the size of the message.Example format:
<tag> (<scop>): <message>
Example usage:
Remember, the first summary or excerpt line should be limited to 50 chars. The following lines can be how ever long.
Current Guidelines are located here: https://cordova.apache.org/contribute/contribute_guidelines.html
Final Notes
This ticket is to only:
I am not trying to resolve, change, or enforced some guideline immediately so please do not try push this through ASAP.
At of the time of this posting, I am heavily focused on preparing major releases and hoping to get them out. If someone didn't follow the current or proposed guideline, I do not see any harm has it has not been very strict in the recent years. It current goal is to not block contribution or the major release process. The merger also has a chance to change the commit message when
squash & merge
.I don't want to take too much time or effort away form the major release right now.
The text was updated successfully, but these errors were encountered: