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
Good git commit message practice is to write the message such that it fits into this template: "If I apply this commit, it will (...)"
Example:
"If I add this commit, it will 'pkg: Add at v(something)'" (the prefix is to make it easier to see which recipe a change belongs to at a glance in a monorepo context).
vs.
"If I add this commit, it will 'pkg: Initial inclusion at v(something)'" (now it's even less correct than before)
Additionally, the git commit summary as displayed by the worklog.py script has (I believe) 52 characters before git begins truncating the commit summary. "Add at" is six characters (leaving 46 characters for the package name and the description) whereas "Initial inclusion at" is twenty characters, leaving 32 characters before git begins truncating the commit summary.
The text was updated successfully, but these errors were encountered:
Any chance we could guide people to use "Add at" instead of "Initial inclusion at" via the git hooks / CI scripts?
I could be persuaded to allow "Include at" at a stretch if "Include" is really that important to people.
But "Initial inclusion" just hertz my soul due to its frequency of appearance... >_>
cf. https://help.getsol.us/docs/packaging/creating-a-new-package#commit-message-format-for-new-packages
Context
Good git commit message practice is to write the message such that it fits into this template: "If I apply this commit, it will (...)"
Example:
"If I add this commit, it will 'pkg: Add at v(something)'" (the prefix is to make it easier to see which recipe a change belongs to at a glance in a monorepo context).
vs.
"If I add this commit, it will 'pkg: Initial inclusion at v(something)'" (now it's even less correct than before)
Additionally, the git commit summary as displayed by the worklog.py script has (I believe) 52 characters before git begins truncating the commit summary. "Add at" is six characters (leaving 46 characters for the package name and the description) whereas "Initial inclusion at" is twenty characters, leaving 32 characters before git begins truncating the commit summary.
The text was updated successfully, but these errors were encountered: