Skip to content

Commit

Permalink
fix: semver.diff prerelease to release recognition
Browse files Browse the repository at this point in the history
  • Loading branch information
dominique-blockchain authored and wraithgar committed Apr 5, 2023
1 parent da08e01 commit b3a7d63
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 4 deletions.
13 changes: 13 additions & 0 deletions functions/diff.js
Expand Up @@ -17,6 +17,19 @@ const diff = (version1, version2) => {
}
}
}
if (v1.major === v2.major &&
v1.minor === v2.minor &&
v1.patch === v2.patch && !(v1.prerelease.length && v2.prerelease.length)) {
const releaseTypes = []
for (const key in v1) {
if (key === 'major' || key === 'minor' || key === 'patch') {
if (v1[key]) {
releaseTypes.push(key)
}
}
}
return releaseTypes.pop()
}
return defaultResult // may be undefined
}
}
Expand Down
14 changes: 10 additions & 4 deletions test/functions/diff.js
Expand Up @@ -5,18 +5,24 @@ test('diff versions test', (t) => {
// [version1, version2, result]
// diff(version1, version2) -> result
[['1.2.3', '0.2.3', 'major'],
['0.2.3', '1.2.3', 'major'],
['1.4.5', '0.2.3', 'major'],
['1.2.3', '2.0.0-pre', 'premajor'],
['2.0.0-pre', '1.2.3', 'premajor'],
['1.2.3', '1.3.3', 'minor'],
['1.0.1', '1.1.0-pre', 'preminor'],
['1.2.3', '1.2.4', 'patch'],
['1.2.3', '1.2.4-pre', 'prepatch'],
['0.0.1', '0.0.1-pre', 'prerelease'],
['0.0.1', '0.0.1-pre-2', 'prerelease'],
['1.1.0', '1.1.0-pre', 'prerelease'],
['0.0.1', '0.0.1-pre', 'patch'],
['0.0.1', '0.0.1-pre-2', 'patch'],
['1.1.0', '1.1.0-pre', 'minor'],
['1.1.0-pre-1', '1.1.0-pre-2', 'prerelease'],
['1.0.0', '1.0.0', null],

['1.0.0', '1.0.0', null],
['1.0.0', '1.0.0', null],
['0.0.2-1', '0.0.2', 'patch'],
['0.1.0-1', '0.1.0', 'minor'],
['1.0.0-1', '1.0.0', 'major'],
].forEach((v) => {
const version1 = v[0]
const version2 = v[1]
Expand Down

0 comments on commit b3a7d63

Please sign in to comment.