fix: use full commit hashes for links

qoomon · qoomon · commit 5901fc553adc · 2023-03-02T21:12:02.000+01:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,10 @@
+## **2.6.3**&emsp;<sub><sup>2023-03-02 ([ad1d56c...a64d54f](https://github.com/qoomon/git-conventional-commits/compare/ad1d56c...a64d54f?diff=split))</sup></sub>
+
+### Bug Fixes
+* use full commit hashes for links ([a64d54f](https://github.com/qoomon/git-conventional-commits/commit/a64d54f))
+
+<br>
+
 ## **2.6.0**&emsp;<sub><sup>2023-02-28 ([aafcc05...aafcc05](https://github.com/qoomon/git-conventional-commits/compare/aafcc05...aafcc05?diff=split))</sup></sub>
 
 ### Features
diff --git a/lib/changelogGenerator.js b/lib/changelogGenerator.js
@@ -227,8 +227,9 @@ module.exports = function (config) {
 
     function markdownCommitRange(formCommitHash, toCommitHash) {
         if (config.commitRangeUrl) {
-            const commitRangeUrl = config.commitRangeUrl.replace("%from%", formCommitHash).replace("%to%",
-                toCommitHash);
+            const commitRangeUrl = config.commitRangeUrl
+                .replace("%from%", formCommitHash)
+                .replace("%to%", toCommitHash);
             return markdownLink(`${formCommitHash.substring(0, 7)}...${toCommitHash.substring(0, 7)}`, commitRangeUrl);
         }
 
diff --git a/lib/git.js b/lib/git.js
@@ -29,7 +29,7 @@ const LOG_COMMIT_DELIMITER = '===LOG_COMMIT_DELIMITER===';
 const LOG_FIELD_SEPARATOR = '===LOG_FIELD_SEPARATOR===';
 
 async function getCommitLog(from, to = 'HEAD') {
-    const gitLogFormat = ["%h", "%aI", "%s", "%b"].join(LOG_FIELD_SEPARATOR) + LOG_COMMIT_DELIMITER;
+    const gitLogFormat = ["%H", "%aI", "%s", "%b"].join(LOG_FIELD_SEPARATOR) + LOG_COMMIT_DELIMITER;
     const gitLog = await execAsync(`git log --reverse --format=${gitLogFormat} ${from ? `${from}..` : ''}${to}`, {encoding: 'UTF-8'})
         .then(result => result.split(LOG_COMMIT_DELIMITER + '\n').slice(0, -1))
         .catch(err => {
diff --git a/lib/gitCommitConvention.js b/lib/gitCommitConvention.js
@@ -102,7 +102,7 @@ module.exports = function (convention, commitAnchor = 'HEAD') {
                     conventionalSubject = parseCommitSubject(conventionalSubject);
                     conventionalSubject.revert = !conventionalSubject.revert; // negate revert of revert commit
                 } else {
-                    console.warn(`[WARN] ${commit.hash ? `${commit.hash} - ` : ''}Invalid commit subject format: '${commit.subject}'`);
+                    console.warn(`[WARN] ${commit.hash ? `${commit.hash.substring(0, 7)} - ` : ''}Invalid commit subject format: '${commit.subject}'`);
                     return conventionalSubject;
                 }
             }
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "git-conventional-commits",
-  "version": "2.6.2",
+  "version": "2.6.3",
   "description": "git conventional commits util",
   "licence": "GPLv3",
   "main": "cli.js",

Original file line number	Diff line number	Diff line change
`@@ -102,7 +102,7 @@ module.exports = function (convention, commitAnchor = 'HEAD') {`
`102`	`102`	`conventionalSubject = parseCommitSubject(conventionalSubject);`
`103`	`103`	`conventionalSubject.revert = !conventionalSubject.revert; // negate revert of revert commit`
`104`	`104`	`} else {`
`105`		- console.warn(`[WARN] ${commit.hash ? `${commit.hash} - ` : ''}Invalid commit subject format: '${commit.subject}'`);
	`105`	+ console.warn(`[WARN] ${commit.hash ? `${commit.hash.substring(0, 7)} - ` : ''}Invalid commit subject format: '${commit.subject}'`);
`106`	`106`	`return conventionalSubject;`
`107`	`107`	`}`
`108`	`108`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "git-conventional-commits",`
`3`		`- "version": "2.6.2",`
	`3`	`+ "version": "2.6.3",`
`4`	`4`	`"description": "git conventional commits util",`
`5`	`5`	`"licence": "GPLv3",`
`6`	`6`	`"main": "cli.js",`