feat: allow identifierBase to be false

README.md

@@ -239,20 +239,40 @@ $ semver 1.2.4-beta.0 -i prerelease
 
 The method `.inc` takes an optional parameter 'identifierBase' string
 that will let you let your prerelease number as zero-based or one-based.
+If you provide false zero will be omitted.
 If you do not specify this parameter, it will default to zero-based.
 
 ```javascript
 semver.inc('1.2.3', 'prerelease', 'beta', '1')
 // '1.2.4-beta.1'
 ```
 
+```javascript
+semver.inc('1.2.3', 'prerelease', 'beta', false)
+// '1.2.4-beta'
+```
+
 command-line example:
 
 ```bash
 $ semver 1.2.3 -i prerelease --preid beta -n 1
 1.2.4-beta.1
 ```
 
+```bash
+$ semver 1.2.3 -i prerelease --preid beta -n false
+1.2.4-beta
+```
+
+
+if another version is created with same identifier then the prerelease
+number is used based on Prerelease Identifier Base
+
+```javascript
+semver.inc('1.2.3-beta', 'prerelease', { disableIdentifierBase: true } , 'beta', '1')
+// '1.2.3-beta.1'
+```
+
 ### Advanced Range Syntax
 
 Advanced range syntax desugars to primitive comparators in

bin/semver.js

@@ -178,6 +178,13 @@ Options:
 --ltr
         Coerce version strings left to right (default)
 
+-n <base>
+        Prerelease Identifier Base
+        that will let you let your prerelease number as 
+        zero-based or one-based. If you provide false
+        zero will be omitted. If you do not specify 
+        this parameter, it will default to zero-based.
+
 Program exits successfully if any valid version satisfies
 all supplied ranges, and prints all satisfying versions.
 

classes/semver.js

@@ -246,9 +246,13 @@ class SemVer {
         break
       // This probably shouldn't be used publicly.
       // 1.0.0 'pre' would become 1.0.0-0 which is the wrong direction.
-      case 'pre':
+      case 'pre': {
+        const base = identifierBase === false ||
+        identifierBase === 'false' ||
+        Number(identifierBase) ? 1 : 0
+
         if (this.prerelease.length === 0) {
-          this.prerelease = [0]
+          this.prerelease = [base]
         } else {
           let i = this.prerelease.length
           while (--i >= 0) {
@@ -259,23 +263,26 @@ class SemVer {
           }
           if (i === -1) {
             // didn't increment anything
-            this.prerelease.push(0)
+            this.prerelease.push(base)
           }
         }
         if (identifier) {
-          const base = Number(identifierBase) ? 1 : 0
           // 1.2.0-beta.1 bumps to 1.2.0-beta.2,
           // 1.2.0-beta.fooblz or 1.2.0-beta bumps to 1.2.0-beta.0
+          let prerelease = [identifier, base]
+          if (identifierBase === false || identifierBase === 'false') {
+            prerelease = [identifier]
+          }
           if (compareIdentifiers(this.prerelease[0], identifier) === 0) {
             if (isNaN(this.prerelease[1])) {
-              this.prerelease = [identifier, base]
+              this.prerelease = prerelease
             }
           } else {
-            this.prerelease = [identifier, base]
+            this.prerelease = prerelease
           }
         }
         break
-
+      }
       default:
         throw new Error(`invalid increment argument: ${release}`)
     }

tap-snapshots/test/bin/semver.js.test.cjs

@@ -93,6 +93,13 @@ Object {
     --ltr
             Coerce version strings left to right (default)
 
+    -n <base>
+            Prerelease Identifier Base
+            that will let you let your prerelease number as 
+            zero-based or one-based. If you provide false
+            zero will be omitted. If you do not specify 
+            this parameter, it will default to zero-based.
+
     Program exits successfully if any valid version satisfies
     all supplied ranges, and prints all satisfying versions.
 
@@ -149,6 +156,13 @@ Object {
     --ltr
             Coerce version strings left to right (default)
 
+    -n <base>
+            Prerelease Identifier Base
+            that will let you let your prerelease number as 
+            zero-based or one-based. If you provide false
+            zero will be omitted. If you do not specify 
+            this parameter, it will default to zero-based.
+
     Program exits successfully if any valid version satisfies
     all supplied ranges, and prints all satisfying versions.
 
@@ -205,6 +219,13 @@ Object {
     --ltr
             Coerce version strings left to right (default)
 
+    -n <base>
+            Prerelease Identifier Base
+            that will let you let your prerelease number as 
+            zero-based or one-based. If you provide false
+            zero will be omitted. If you do not specify 
+            this parameter, it will default to zero-based.
+
     Program exits successfully if any valid version satisfies
     all supplied ranges, and prints all satisfying versions.
 
@@ -261,6 +282,13 @@ Object {
     --ltr
             Coerce version strings left to right (default)
 
+    -n <base>
+            Prerelease Identifier Base
+            that will let you let your prerelease number as 
+            zero-based or one-based. If you provide false
+            zero will be omitted. If you do not specify 
+            this parameter, it will default to zero-based.
+
     Program exits successfully if any valid version satisfies
     all supplied ranges, and prints all satisfying versions.
 
@@ -310,6 +338,15 @@ Object {
 }
 `
 
+exports[`test/bin/semver.js TAP inc tests > -i premajor 1.0.0 --preid=beta -n false 1`] = `
+Object {
+  "code": 0,
+  "err": "",
+  "out": "2.0.0-beta\\n",
+  "signal": null,
+}
+`
+
 exports[`test/bin/semver.js TAP inc tests > -i premajor 1.0.0 --preid=beta 1`] = `
 Object {
   "code": 0,

test/bin/semver.js

@@ -28,8 +28,8 @@ const run = args => new Promise((resolve, reject) => {
 t.test('inc tests', t => Promise.all([
   ['-i', 'major', '1.0.0'],
   ['-i', 'major', '1.0.0', '1.0.1'],
-  ['-i', 'premajor', '1.0.0', '--preid=beta'],
   ['-i', 'premajor', '1.0.0', '--preid=beta', '-n', '1'],
+  ['-i', 'premajor', '1.0.0', '--preid=beta', '-n', 'false'],
   ['-i', '1.2.3'],
 ].map(args => t.resolveMatchSnapshot(run(args), args.join(' ')))))
 

test/fixtures/increments.js

@@ -101,4 +101,19 @@ module.exports = [
   ['1.2.3-dev.bar', 'prerelease', '1.2.3-dev.0', false, 'dev', '0'],
   ['1.2.0', 'preminor', '1.3.0-dev.1', false, 'dev', '1'],
   ['1.2.3-1', 'preminor', '1.3.0-dev.0', false, 'dev'],
+
+  ['1.2.0-1', 'prerelease', '1.2.0-alpha', false, 'alpha', false],
+  ['1.2.1', 'prerelease', '1.2.2-alpha', false, 'alpha', false],
+  ['1.2.2', 'prerelease', '1.2.3-alpha', false, 'alpha', false],
+  ['1.2.0', 'prepatch', '1.2.1-dev', false, 'dev', false],
+  ['1.2.0-1', 'prepatch', '1.2.1-dev', false, 'dev', false],
+  ['1.2.0', 'premajor', '2.0.0-dev', false, 'dev', false],
+  ['1.2.3-1', 'premajor', '2.0.0-dev', false, 'dev', false],
+  ['1.2.3-dev.bar', 'prerelease', '1.2.3-dev', false, 'dev', false],
+  ['1.2.0', 'preminor', '1.3.0-dev', false, 'dev', false],
+  ['1.2.3-1', 'preminor', '1.3.0-dev', false, 'dev', false],
+  ['1.2.3-dev', 'prerelease', '1.2.3-dev.1', false, 'dev', false],
+  ['1.2.0-dev', 'premajor', '2.0.0-dev', false, 'dev', false],
+  ['1.2.0-dev', 'preminor', '1.3.0-beta', false, 'beta', false],
+  ['1.2.0-dev', 'prepatch', '1.2.1-dev', false, 'dev', false],
 ]