Merge pull request #2 from beyarkay/fix-asset-uploads

Fix: Asset uploads fail if >30 assets #1
IsaacShelton · Jul 18, 2022 · 3d526d3 · 3d526d3
2 parents 4881928 + ddf6001
commit 3d526d3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 9 deletions.
diff --git a/dist/main.js b/dist/main.js
diff --git a/src/main.ts b/src/main.ts
@@ -208,7 +208,7 @@ class Connection {
             }
 
             if (shouldDelete) {
-                core.startGroup('Deleting old release asset id ' + asset.id + '...');
+                core.startGroup('Deleting old release asset ' + asset.name + ' (' + asset.id + ')...');
                 await this.github.rest.repos.deleteReleaseAsset(
                     {
                         ...context.repo,
@@ -238,17 +238,11 @@ class Connection {
         if (this.id < 0)
             return;
 
-        let assets = [];
-
-        let response = await this.github.paginate(this.github.rest.repos.listReleaseAssets, {
+        let assets = await this.github.paginate(this.github.rest.repos.listReleaseAssets, {
             ...context.repo,
             release_id: this.id,
         });
 
-        for(let asset of response){
-             assets.push(asset);
-        }
-
         this.dump('assets', assets);
         core.endGroup();
         return assets;