Allow Infinity as line limit (#137)

GitHub started reflowing the descriptions of the pull requests, so we
want often to disable the check in the pull requests.

Author: mristin

.github/workflows/build-and-test.yml

@@ -27,6 +27,7 @@ jobs:
         with:
           additional-verbs: 'chrusimusi, unit-test'
           path-to-additional-verbs: src/additional-verbs.txt
+          max-body-line-length: 'Infinity'
 
 #  test-pr-commits:
 #    runs-on: ubuntu-latest
@@ -44,3 +45,4 @@ jobs:
       - uses: ./
         with:
           allow-one-liners: 'true'
+          max-body-line-length: 'Infinity'

README.md

@@ -174,6 +174,8 @@ You can change the imposed maximum subject length by setting the flag `max-subje
           max-subject-line-length: '100'
 ```
 
+If you want to disable the limit, set it to `Infinity`.
+
 ## Custom line length on the body
 
 Similar to the subject line, for terminals and monospaced GUIs it is a good practice to limit the line length of the body to 72 characters.
@@ -190,6 +192,8 @@ You can change the imposed maximum line length by setting the flag `max-body-lin
           max-body-line-length: '100'
 ```
 
+If you want to disable the limit, set it to `Infinity`.
+
 ## Skip Body Check
 
 For some repositories only the subject matters while the body is allowed to be free-form.

dist/index.js

@@ -29058,6 +29058,25 @@ class MaybeInputs {
     }
 }
 exports.MaybeInputs = MaybeInputs;
+const infLiteralSet = new Set([
+    'inf',
+    'infty',
+    'infinity',
+    '-inf',
+    '-infty',
+    '-infinity',
+]);
+/**
+ * Parse the `text` as either an integer or `Infinity`.
+ *
+ * If the `text` could not be parsed, return a `NaN`.
+ */
+function parseIntOrInfinity(text) {
+    if (infLiteralSet.has(text.toLowerCase())) {
+        return Infinity;
+    }
+    return parseInt(text, 10);
+}
 function parseInputs(rawInputs) {
     const { additionalVerbsInput = '', pathToAdditionalVerbsInput = '', allowOneLinersInput = '', maxSubjectLengthInput = '', maxBodyLineLengthInput = '', enforceSignOffInput = '', validatePullRequestCommitsInput = '', skipBodyCheckInput = '', ignoreMergeCommitsInput = '', ignorePatternsInput = '', } = rawInputs;
     const additionalVerbs = new Set();
@@ -29086,14 +29105,14 @@ function parseInputs(rawInputs) {
     }
     const maxSubjectLength = !maxSubjectLengthInput
         ? 50
-        : parseInt(maxSubjectLengthInput, 10);
+        : parseIntOrInfinity(maxSubjectLengthInput);
     if (Number.isNaN(maxSubjectLength)) {
         return new MaybeInputs(null, 'Unexpected value for max-subject-line-length. ' +
             `Expected a number or nothing, got ${maxSubjectLengthInput}`);
     }
     const maxBodyLineLength = !maxBodyLineLengthInput
         ? 72
-        : parseInt(maxBodyLineLengthInput, 10);
+        : parseIntOrInfinity(maxBodyLineLengthInput);
     if (Number.isNaN(maxBodyLineLength)) {
         return new MaybeInputs(null, 'Unexpected value for max-body-line-length. ' +
             `Expected a number or nothing, got ${maxBodyLineLengthInput}`);

package-lock.json

@@ -43,7 +43,7 @@ it('parses the inputs.', () => {
     ignorePatternsInput: `
     ^Some pattern$
     Another pattern
-    `
+    `,
   });
 
   expect(maybeInputs.error).toBeNull();
@@ -52,7 +52,7 @@ it('parses the inputs.', () => {
   expect(inputs.hasAdditionalVerbsInput).toBeTruthy();
   expect(inputs.pathToAdditionalVerbs).toEqual(pathToVerbs);
   expect(inputs.additionalVerbs).toEqual(
-    new Set<string>(['rewrap', 'table', 'integrate', 'analyze'])
+    new Set<string>(['rewrap', 'table', 'integrate', 'analyze']),
   );
   expect(inputs.allowOneLiners).toBeTruthy();
   expect(inputs.maxSubjectLength).toEqual(90);
@@ -61,8 +61,18 @@ it('parses the inputs.', () => {
   expect(inputs.validatePullRequestCommits).toBeTruthy();
   expect(inputs.skipBodyCheck).toBeTruthy();
   expect(inputs.ignoreMergeCommits).toBeFalsy();
-  expect(inputs.ignorePatterns).toEqual([
-    /^Some pattern$/,
-    /Another pattern/
-  ]);
+  expect(inputs.ignorePatterns).toEqual([/^Some pattern$/, /Another pattern/]);
+});
+
+it('parses the Infinity limits.', () => {
+  const maybeInputs = input.parseInputs({
+    maxSubjectLengthInput: 'Infinity',
+    maxBodyLineLengthInput: 'Infinity',
+  });
+
+  expect(maybeInputs.error).toBeNull();
+
+  const inputs = maybeInputs.mustInputs();
+  expect(inputs.maxSubjectLength).toEqual(Infinity);
+  expect(inputs.maxBodyLineLength).toEqual(Infinity);
 });