Optimized autoloader in production builds

skaut · Aug 25, 2021 · 10b6291 · 10b6291
1 parent e04500e
commit 10b6291
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 5 deletions.
diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml
@@ -49,7 +49,7 @@ jobs:
 
       - name: "Install Composer dependencies"
         run: |
-          composer install -n --prefer-dist
+          composer -n install
 
       - name: "Cache NPM dependencies"
         uses: actions/[email protected]
@@ -119,7 +119,7 @@ jobs:
 
       - name: "Install Composer dependencies"
         run: |
-          composer install -n --prefer-dist
+          composer -n install
 
       - name: "Cache NPM dependencies"
         uses: actions/[email protected]
@@ -178,7 +178,7 @@ jobs:
 
       - name: "Install Composer dependencies"
         run: |
-          composer install -n --prefer-dist
+          composer -n install
 
       - name: "Cache NPM dependencies"
         uses: actions/[email protected]

diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -32,7 +32,7 @@ jobs:
 
       - name: "Install Composer dependencies"
         run: |
-          composer install -n --prefer-dist
+          composer -n install
 
       - name: "Cache NPM dependencies"
         uses: actions/[email protected]
@@ -49,6 +49,9 @@ jobs:
           npm ci
 
       - name: "Build"
+        with:
+          env:
+            NODE_ENV: 'production'
         run: |
           npm run build
 

diff --git a/gulpfile.js b/gulpfile.js
@@ -43,7 +43,10 @@ gulp.task(
 gulp.task(
 	'build:deps:composer:autoloader',
 	gulp.series(
-		shell.task( 'composer dump-autoload --no-dev' ),
+		shell.task(
+			'composer dump-autoload --no-dev' +
+				( process.env.NODE_ENV === 'production' ? ' -o' : '' )
+		),
 		function () {
 			return merge(
 				gulp.src( [