Remove repo param support & logs

This commit is contained in:
Nishtha Gupta
2022-12-29 15:51:31 +05:30
parent 64305dab94
commit 9dbafb83da
7 changed files with 0 additions and 30 deletions

View File

@@ -89,7 +89,6 @@ describe.skip('index tests -- call graphql', () => {
const defaultInput: InputParams = { const defaultInput: InputParams = {
packageVersionIds: [], packageVersionIds: [],
owner: 'namratajha', owner: 'namratajha',
repo: 'only-pkg',
packageName: 'only-pkg', packageName: 'only-pkg',
numOldVersionsToDelete: 1, numOldVersionsToDelete: 1,
minVersionsToKeep: -1, minVersionsToKeep: -1,

12
dist/index.js vendored
View File

@@ -36,13 +36,10 @@ function finalIds(input) {
return getVersionIds(input.owner, input.packageName, input.packageType, RATE_LIMIT, 1, input.token).pipe( return getVersionIds(input.owner, input.packageName, input.packageType, RATE_LIMIT, 1, input.token).pipe(
// This code block executes on all versions of a package starting from oldest // This code block executes on all versions of a package starting from oldest
operators_2.map(value => { operators_2.map(value => {
console.log('If block');
console.log(`value: ${JSON.stringify(value)}`);
// we need to delete oldest versions first // we need to delete oldest versions first
value.sort((a, b) => { value.sort((a, b) => {
return (new Date(a.created_at).getTime() - new Date(b.created_at).getTime()); return (new Date(a.created_at).getTime() - new Date(b.created_at).getTime());
}); });
console.log(`sorted value: ${JSON.stringify(value)}`);
/* /*
Here first filter out the versions that are to be ignored. Here first filter out the versions that are to be ignored.
Then update input.numOldeVersionsToDelete to the no of versions deleted from the next 100 versions batch. Then update input.numOldeVersionsToDelete to the no of versions deleted from the next 100 versions batch.
@@ -55,7 +52,6 @@ function finalIds(input) {
else { else {
toDelete = Math.min(value.length - input.minVersionsToKeep, RATE_LIMIT); toDelete = Math.min(value.length - input.minVersionsToKeep, RATE_LIMIT);
} }
console.log(`toDelete is ${toDelete}`);
if (toDelete < 0) if (toDelete < 0)
return []; return [];
return value.map(info => info.id.toString()).slice(0, toDelete); return value.map(info => info.id.toString()).slice(0, toDelete);
@@ -93,7 +89,6 @@ exports.Input = void 0;
const defaultParams = { const defaultParams = {
packageVersionIds: [], packageVersionIds: [],
owner: '', owner: '',
repo: '',
packageName: '', packageName: '',
packageType: '', packageType: '',
numOldVersionsToDelete: 0, numOldVersionsToDelete: 0,
@@ -107,7 +102,6 @@ class Input {
const validatedParams = Object.assign(Object.assign({}, defaultParams), params); const validatedParams = Object.assign(Object.assign({}, defaultParams), params);
this.packageVersionIds = validatedParams.packageVersionIds; this.packageVersionIds = validatedParams.packageVersionIds;
this.owner = validatedParams.owner; this.owner = validatedParams.owner;
this.repo = validatedParams.repo;
this.packageName = validatedParams.packageName; this.packageName = validatedParams.packageName;
this.packageType = validatedParams.packageType; this.packageType = validatedParams.packageType;
this.numOldVersionsToDelete = validatedParams.numOldVersionsToDelete; this.numOldVersionsToDelete = validatedParams.numOldVersionsToDelete;
@@ -119,7 +113,6 @@ class Input {
} }
hasOldestVersionQueryInfo() { hasOldestVersionQueryInfo() {
return !!(this.owner && return !!(this.owner &&
this.repo &&
this.packageName && this.packageName &&
this.numOldVersionsToDelete >= 0 && this.numOldVersionsToDelete >= 0 &&
this.token); this.token);
@@ -181,8 +174,6 @@ function deletePackageVersion(packageVersionId, owner, packageName, packageType,
} }
exports.deletePackageVersion = deletePackageVersion; exports.deletePackageVersion = deletePackageVersion;
function deletePackageVersions(packageVersionIds, owner, packageName, packageType, token) { function deletePackageVersions(packageVersionIds, owner, packageName, packageType, token) {
console.log(`Total versions to delete: ${packageVersionIds.length}`);
console.log(`Versions to delete: ${packageVersionIds}`);
if (packageVersionIds.length === 0) { if (packageVersionIds.length === 0) {
return rxjs_1.of(true); return rxjs_1.of(true);
} }
@@ -239,7 +230,6 @@ function getOldestVersions(owner, packageName, packageType, numVersions, page, t
paginate: response.data.length === numVersions, paginate: response.data.length === numVersions,
totalCount: response.data.length totalCount: response.data.length
}; };
console.log(`Response from getOldestVersions method: ${JSON.stringify(resp)}`);
return resp; return resp;
})); }));
} }
@@ -43910,13 +43900,11 @@ const rxjs_1 = __nccwpck_require__(5805);
const delete_1 = __nccwpck_require__(9645); const delete_1 = __nccwpck_require__(9645);
const operators_1 = __nccwpck_require__(7801); const operators_1 = __nccwpck_require__(7801);
function getActionInput() { function getActionInput() {
console.log('this is deletion using rest APIs');
return new input_1.Input({ return new input_1.Input({
packageVersionIds: core_1.getInput('package-version-ids') packageVersionIds: core_1.getInput('package-version-ids')
? core_1.getInput('package-version-ids').split(',') ? core_1.getInput('package-version-ids').split(',')
: [], : [],
owner: core_1.getInput('owner') ? core_1.getInput('owner') : github_1.context.repo.owner, owner: core_1.getInput('owner') ? core_1.getInput('owner') : github_1.context.repo.owner,
repo: core_1.getInput('repo') ? core_1.getInput('repo') : github_1.context.repo.repo,
packageName: core_1.getInput('package-name'), packageName: core_1.getInput('package-name'),
packageType: core_1.getInput('package-type'), packageType: core_1.getInput('package-type'),
numOldVersionsToDelete: Number(core_1.getInput('num-old-versions-to-delete')), numOldVersionsToDelete: Number(core_1.getInput('num-old-versions-to-delete')),

View File

@@ -61,15 +61,12 @@ export function finalIds(input: Input): Observable<string[]> {
).pipe( ).pipe(
// This code block executes on all versions of a package starting from oldest // This code block executes on all versions of a package starting from oldest
map(value => { map(value => {
console.log('If block')
console.log(`value: ${JSON.stringify(value)}`)
// we need to delete oldest versions first // we need to delete oldest versions first
value.sort((a, b) => { value.sort((a, b) => {
return ( return (
new Date(a.created_at).getTime() - new Date(b.created_at).getTime() new Date(a.created_at).getTime() - new Date(b.created_at).getTime()
) )
}) })
console.log(`sorted value: ${JSON.stringify(value)}`)
/* /*
Here first filter out the versions that are to be ignored. Here first filter out the versions that are to be ignored.
Then update input.numOldeVersionsToDelete to the no of versions deleted from the next 100 versions batch. Then update input.numOldeVersionsToDelete to the no of versions deleted from the next 100 versions batch.
@@ -87,7 +84,6 @@ export function finalIds(input: Input): Observable<string[]> {
RATE_LIMIT RATE_LIMIT
) )
} }
console.log(`toDelete is ${toDelete}`)
if (toDelete < 0) return [] if (toDelete < 0) return []
return value.map(info => info.id.toString()).slice(0, toDelete) return value.map(info => info.id.toString()).slice(0, toDelete)
}) })

View File

@@ -1,7 +1,6 @@
export interface InputParams { export interface InputParams {
packageVersionIds?: string[] packageVersionIds?: string[]
owner?: string owner?: string
repo?: string
packageName?: string packageName?: string
packageType?: string packageType?: string
numOldVersionsToDelete?: number numOldVersionsToDelete?: number
@@ -14,7 +13,6 @@ export interface InputParams {
const defaultParams = { const defaultParams = {
packageVersionIds: [], packageVersionIds: [],
owner: '', owner: '',
repo: '',
packageName: '', packageName: '',
packageType: '', packageType: '',
numOldVersionsToDelete: 0, numOldVersionsToDelete: 0,
@@ -27,7 +25,6 @@ const defaultParams = {
export class Input { export class Input {
packageVersionIds: string[] packageVersionIds: string[]
owner: string owner: string
repo: string
packageName: string packageName: string
packageType: string packageType: string
numOldVersionsToDelete: number numOldVersionsToDelete: number
@@ -42,7 +39,6 @@ export class Input {
this.packageVersionIds = validatedParams.packageVersionIds this.packageVersionIds = validatedParams.packageVersionIds
this.owner = validatedParams.owner this.owner = validatedParams.owner
this.repo = validatedParams.repo
this.packageName = validatedParams.packageName this.packageName = validatedParams.packageName
this.packageType = validatedParams.packageType this.packageType = validatedParams.packageType
this.numOldVersionsToDelete = validatedParams.numOldVersionsToDelete this.numOldVersionsToDelete = validatedParams.numOldVersionsToDelete
@@ -56,7 +52,6 @@ export class Input {
hasOldestVersionQueryInfo(): boolean { hasOldestVersionQueryInfo(): boolean {
return !!( return !!(
this.owner && this.owner &&
this.repo &&
this.packageName && this.packageName &&
this.numOldVersionsToDelete >= 0 && this.numOldVersionsToDelete >= 0 &&
this.token this.token

View File

@@ -7,13 +7,11 @@ import {deleteVersions} from './delete'
import {catchError} from 'rxjs/operators' import {catchError} from 'rxjs/operators'
function getActionInput(): Input { function getActionInput(): Input {
console.log('this is deletion using rest APIs')
return new Input({ return new Input({
packageVersionIds: getInput('package-version-ids') packageVersionIds: getInput('package-version-ids')
? getInput('package-version-ids').split(',') ? getInput('package-version-ids').split(',')
: [], : [],
owner: getInput('owner') ? getInput('owner') : context.repo.owner, owner: getInput('owner') ? getInput('owner') : context.repo.owner,
repo: getInput('repo') ? getInput('repo') : context.repo.repo,
packageName: getInput('package-name'), packageName: getInput('package-name'),
packageType: getInput('package-type'), packageType: getInput('package-type'),
numOldVersionsToDelete: Number(getInput('num-old-versions-to-delete')), numOldVersionsToDelete: Number(getInput('num-old-versions-to-delete')),

View File

@@ -48,9 +48,6 @@ export function deletePackageVersions(
packageType: string, packageType: string,
token: string token: string
): Observable<boolean> { ): Observable<boolean> {
console.log(`Total versions to delete: ${packageVersionIds.length}`)
console.log(`Versions to delete: ${packageVersionIds}`)
if (packageVersionIds.length === 0) { if (packageVersionIds.length === 0) {
return of(true) return of(true)
} }

View File

@@ -63,9 +63,6 @@ export function getOldestVersions(
paginate: response.data.length === numVersions, paginate: response.data.length === numVersions,
totalCount: response.data.length totalCount: response.data.length
} }
console.log(
`Response from getOldestVersions method: ${JSON.stringify(resp)}`
)
return resp return resp
}) })
) )