fix error for deleting all pre-release versions

This commit is contained in:
Namrata Jha
2021-11-17 13:15:23 +00:00
committed by GitHub
parent 7f40a506a4
commit b5e80e74e4
3 changed files with 6 additions and 6 deletions

5
dist/index.js vendored
View File

@@ -34,7 +34,8 @@ function getVersionIds(input) {
if (input.hasOldestVersionQueryInfo()) {
return version_1.getOldestVersions(input.owner, input.repo, input.packageName, input.numOldVersionsToDelete + input.minVersionsToKeep, input.token).pipe(operators_1.map(versionInfo => {
const numberVersionsToDelete = versionInfo.length - input.minVersionsToKeep;
if (input.deletePreReleaseVersions == 'true') {
if (input.deletePreReleaseVersions == 'true' &&
input.minVersionsToKeep > 0) {
return numberVersionsToDelete <= 0
? []
: versionInfo
@@ -101,8 +102,6 @@ class Input {
this.deletePreReleaseVersions = validatedParams.deletePreReleaseVersions;
this.token = validatedParams.token;
if (this.deletePreReleaseVersions == 'true') {
this.minVersionsToKeep =
this.minVersionsToKeep > 0 ? this.minVersionsToKeep : 1;
this.numOldVersionsToDelete = 100 - this.minVersionsToKeep;
this.ignoreVersions = new RegExp('^(0|[1-9]\\d*)((\\.(0|[1-9]\\d*))*)$');
}

View File

@@ -20,7 +20,10 @@ export function getVersionIds(input: Input): Observable<string[]> {
const numberVersionsToDelete =
versionInfo.length - input.minVersionsToKeep
if (input.deletePreReleaseVersions == 'true') {
if (
input.deletePreReleaseVersions == 'true' &&
input.minVersionsToKeep > 0
) {
return numberVersionsToDelete <= 0
? []
: versionInfo

View File

@@ -47,8 +47,6 @@ export class Input {
this.token = validatedParams.token
if (this.deletePreReleaseVersions == 'true') {
this.minVersionsToKeep =
this.minVersionsToKeep > 0 ? this.minVersionsToKeep : 1
this.numOldVersionsToDelete = 100 - this.minVersionsToKeep
this.ignoreVersions = new RegExp('^(0|[1-9]\\d*)((\\.(0|[1-9]\\d*))*)$')
}