Merge pull request #173 from yacaovsnc/main

Display a warning if user tries to upload over 10000 files
actions · Feb 8, 2021 · ea3d524 · ea3d524
2 parents a1af2e8 + e95d1b9
commit ea3d524
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/dist/index.js b/dist/index.js
@@ -4078,6 +4078,9 @@ function run() {
                 const s = searchResult.filesToUpload.length === 1 ? '' : 's';
                 core.info(`With the provided path, there will be ${searchResult.filesToUpload.length} file${s} uploaded`);
                 core.debug(`Root artifact directory is ${searchResult.rootDirectory}`);
+                if (searchResult.filesToUpload.length > 10000) {
+                    core.warning(`There are over 10,000 files in this artifact, consider create an archive before upload to improve the upload performance.`);
+                }
                 const artifactClient = artifact_1.create();
                 const options = {
                     continueOnError: false

diff --git a/src/upload-artifact.ts b/src/upload-artifact.ts
@@ -37,6 +37,12 @@ async function run(): Promise<void> {
       )
       core.debug(`Root artifact directory is ${searchResult.rootDirectory}`)
 
+      if (searchResult.filesToUpload.length > 10000) {
+        core.warning(
+          `There are over 10,000 files in this artifact, consider create an archive before upload to improve the upload performance.`
+        )
+      }
+
       const artifactClient = create()
       const options: UploadOptions = {
         continueOnError: false