Add M/m for merges - Githubissues

RefactoringCombos / ArlosCommitNotation

A notation for small commits messages that show the risk involved in each step

310 stars 30 forks source link

Add M/m for merges #23

Closed JayBazuzi closed 3 years ago

arlobelshee commented 3 years ago

I've been encouraging use of the F or B family for these. That way the commits along the main branch all state the overall features / bugs / progress items that people made, so readers can ignore the side branch unless they want detail. This also gives people a good place to put the overall summary info & the various tracking tags (like Jira ticket numbers, common spec notices, etc - all the things that tie into automation).

Thoughts?

JayBazuzi commented 3 years ago

That could work. Maybe some rule about the max risk of all the sub-commits?

R r r r -> R R R B -> B

arlobelshee commented 3 years ago

I agree on the max risk, but would say that the letter choice should match the intent. Eg

r r R r t B!! t r r F r R r -> F!! (was intending F, did a B along the way, max risk was !!)

arlobelshee commented 3 years ago

I added M as an extended intention when I re-wrote this in terms of risk levels and intentions. Does that meet the needs of this PR, and if so can we close it?

JayBazuzi commented 3 years ago

Yes.