summaryrefslogtreecommitdiffstats
path: root/docs
diff options
context:
space:
mode:
authorNew Tobin Paradigm <email@mattatobin.com>2020-10-02 05:13:43 -0400
committerMoonchild <moonchild@palemoon.org>2020-10-06 09:31:28 +0000
commitb7dac18c0e70c6f0e825ce2a7d44fdaa2cc2ad70 (patch)
tree93c797c7b9ec3d9ab5b46c635697b681414052fb /docs
parente50094ab02604892ad619abb0e3d1c7c674e43cf (diff)
downloadUXP-b7dac18c0e70c6f0e825ce2a7d44fdaa2cc2ad70.tar
UXP-b7dac18c0e70c6f0e825ce2a7d44fdaa2cc2ad70.tar.gz
UXP-b7dac18c0e70c6f0e825ce2a7d44fdaa2cc2ad70.tar.lz
UXP-b7dac18c0e70c6f0e825ce2a7d44fdaa2cc2ad70.tar.xz
UXP-b7dac18c0e70c6f0e825ce2a7d44fdaa2cc2ad70.zip
Update CONTRIBUTING.md
Revise Commit message style and add Follow-up form
Diffstat (limited to 'docs')
-rw-r--r--docs/CONTRIBUTING.md9
1 files changed, 7 insertions, 2 deletions
diff --git a/docs/CONTRIBUTING.md b/docs/CONTRIBUTING.md
index 9379ea45a..031ddb904 100644
--- a/docs/CONTRIBUTING.md
+++ b/docs/CONTRIBUTING.md
@@ -93,15 +93,20 @@ With rare exception, it is advisable to use the following style for commit messa
This would directly apply to anyone making a pull request.
+Single commits are where issues can be reasonably resolved in a single commit.
- Single Commit: `Issue #xxx - Cited issue title or appropriate direct description of changes`
- *`Issue #1083 - Deprecate FUEL extension helper javascript library`*
-- Multi-Part Commits: `Issue #xxx - Part Number: Appropriate direct description of changes`
+
+Multi-part commits would be used for complex issues. However, an exception exists for multi-part commits where the issue is anticipated or ends up being long term such as "Stop using unified compilation of sources". In these instances the multi-part form is not required.
+- Multi-Part Form: `Issue #xxx - Part Number: Appropriate direct description of changes`
- *`Issue #492 - Part 1: Remove files`*
- *`Issue #492 - Part 2: Build system, Installer/Packaging`*
- *`Issue #492 - Part 3: nsUpdateService.js, updater.cpp, nsUpdateDriver.cpp`*
- *`Issue #492 - Part 4: Remove superfluous brackets in nsUpdateService.js and updater.cpp`*
-An exception exists for multi-part commits where the issue is anticipated or ends up being long term such as "Stop using unified compilation of sources". In these instances the multi-part form is not required.
+Occasionally a resolved (and shipped) issue requires further changes to fix bugs. However, If follow-ups are complex enough to be multi-part it should be considered a new issue.
+- Follow-up Form: *`Issue #xxx - Follow-up: Appropriate direct description of changes`*
+ - *`Issue #1643 - Follow-up: Make sure things aren't changed while iterating.`*
### Commits with no issue