Fix lint-staged command for package.json files (#918)

seek-oss · Jan 16, 2024 · f77940d · f77940d
1 parent c01f13a
commit f77940d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/.changeset/metal-hairs-look.md b/.changeset/metal-hairs-look.md
@@ -0,0 +1,5 @@
+---
+'sku': patch
+---
+
+Fix running lint-staged when `package.json` files are staged
diff --git a/packages/sku/config/lintStaged/lintStagedConfig.js b/packages/sku/config/lintStaged/lintStagedConfig.js
@@ -11,9 +11,10 @@ const config = {
 
 // Yarn lock integrity check
 if (isYarn) {
-  config['+(package.json|yarn.lock)'] = [
-    getCommand('yarn', 'install', ['--check-files']),
-  ];
+  config['+(package.json|yarn.lock)'] =
+    // The function form allows running the command without file arguments
+    // https://github.com/lint-staged/lint-staged#example-run-tsc-on-changes-to-typescript-files-but-do-not-pass-any-filename-arguments
+    () => getCommand('yarn', 'install', ['--check-files']);
 }
 
 module.exports = config;