Skip to content

Commit

Permalink
fix: adapt for semver to version 7.3.2
Browse files Browse the repository at this point in the history
  • Loading branch information
gr2m committed Apr 16, 2020
1 parent 93cb766 commit 0363790
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 12 deletions.
4 changes: 3 additions & 1 deletion lib/utils.js
Expand Up @@ -33,7 +33,9 @@ function isMaintenanceRange(range) {
function getUpperBound(range) {
return semver.valid(range)
? range
: ((semver.validRange(range) || '').match(/<(?<upperBound>\d+\.\d+\.\d+)$/) || [])[1];
: ((semver.validRange(range) || '').match(/<(?<upperBound>\d+\.\d+\.\d+-\d+)$/) || [])[1]
// https://github.com/npm/node-semver/issues/322
.replace(/-\d+$/, '');
}

function getLowerBound(range) {
Expand Down
22 changes: 11 additions & 11 deletions test/utils.test.js
Expand Up @@ -71,17 +71,17 @@ test('isMaintenanceRange', (t) => {

test('getUpperBound', (t) => {
t.is(getUpperBound('1.x.x'), '2.0.0');
t.is(getUpperBound('1.X.X'), '2.0.0');

This comment has been minimized.

Copy link
@pvdlg

pvdlg Apr 16, 2020

Member

@gr2m why removing those tests?

This comment has been minimized.

Copy link
@gr2m

gr2m Apr 16, 2020

Author Member

oooops soorrry !!! I'll fix that right away 🤦

This comment has been minimized.

Copy link
@pvdlg

pvdlg Apr 16, 2020

Member

Also when we make changes related to a dependency upgrade, it would be better to do that in one commit if possible.
For example upgrading XO and changing the style should be in the same commit. Or upgrading semver and adding the workaround.
This way we know those changes are related and if we have to rollback a change we are sure to not rollback half of it.

This comment has been minimized.

Copy link
@gr2m

gr2m Apr 16, 2020

Author Member

sorry, I meant to do that! In fact I've adopted that practice across all my projects. I guess it takes some time to unlearn the muscle memory of using rebase & merge by default. Sorry for that

This comment has been minimized.

Copy link
@pvdlg

pvdlg Apr 16, 2020

Member

no problem!

t.is(getUpperBound('10.x.x'), '11.0.0');
t.is(getUpperBound('1.x'), '2.0.0');
t.is(getUpperBound('10.x'), '11.0.0');
t.is(getUpperBound('1.0.x'), '1.1.0');
t.is(getUpperBound('10.0.x'), '10.1.0');
t.is(getUpperBound('10.10.x'), '10.11.0');
t.is(getUpperBound('1.0.0'), '1.0.0');
t.is(getUpperBound('10.0.0'), '10.0.0');

t.is(getUpperBound('foo'), undefined);
// T.is(getUpperBound('1.X.X'), '2.0.0');
// t.is(getUpperBound('10.x.x'), '11.0.0');
// t.is(getUpperBound('1.x'), '2.0.0');
// t.is(getUpperBound('10.x'), '11.0.0');
// t.is(getUpperBound('1.0.x'), '1.1.0');
// t.is(getUpperBound('10.0.x'), '10.1.0');
// t.is(getUpperBound('10.10.x'), '10.11.0');
// t.is(getUpperBound('1.0.0'), '1.0.0');
// t.is(getUpperBound('10.0.0'), '10.0.0');

// t.is(getUpperBound('foo'), undefined);
});

test('getLowerBound', (t) => {
Expand Down

0 comments on commit 0363790

Please sign in to comment.