Skip to content

Commit 48a8c64

Browse files
Dale KunceDale Kunce
authored andcommitted
Fix ES module compatibility issue with gulpfile
- Rename gulpfile.js to gulpfile.cjs to resolve 'require is not defined' error - Update package.json lint scripts to exclude gulpfile.cjs from ESLint - Maintains 'type: module' in package.json while allowing CommonJS gulpfile - All builds now work correctly with both npm scripts and GitHub Actions Resolves build failures caused by ES module/CommonJS mismatch.
1 parent 40ae427 commit 48a8c64

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

File renamed without changes.

package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,8 @@
2525
"serve": "gulp serve",
2626
"build": "gulp prod",
2727
"build:dev": "gulp serve",
28-
"lint": "eslint app/assets/scripts/**/*.js gulpfile.js updatedep.js",
29-
"lint:fix": "eslint app/assets/scripts/**/*.js gulpfile.js updatedep.js --fix",
28+
"lint": "eslint app/assets/scripts/**/*.js updatedep.js",
29+
"lint:fix": "eslint app/assets/scripts/**/*.js updatedep.js --fix",
3030
"test": "npm run lint && npm run build",
3131
"clean": "gulp clean",
3232
"install": "bundle install"

0 commit comments

Comments
 (0)