@storm-software/git-tools 1.2.17 → 1.2.18

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/CHANGELOG.md CHANGED
@@ -1,3 +1,13 @@
1
+ ## [1.2.17](https://github.com/storm-software/storm-ops/compare/git-tools-v1.2.16...git-tools-v1.2.17) (2023-11-10)
2
+
3
+
4
+ ### Bug Fixes
5
+
6
+ * **git-tools:** Added logging to husky hooks ([5b4a159](https://github.com/storm-software/storm-ops/commit/5b4a1597fee820bf1aa009473b313d43d06ae1c9))
7
+ * **git-tools:** Moved hook executable scripts to bin folder ([36f8896](https://github.com/storm-software/storm-ops/commit/36f889633a971ac091491dd82fbd40d6cc6236dc))
8
+ * **git-tools:** Updated executable scripts to check for git-lfs failures ([ad738cf](https://github.com/storm-software/storm-ops/commit/ad738cfaadc30306c9ff9e2e7fde19c9b328ccaf))
9
+ * **git-tools:** Wrapped all hook executable scripts with try-catches ([f36b181](https://github.com/storm-software/storm-ops/commit/f36b1813a88970eb7fd24c67ef71e93bca7edcf9))
10
+
1
11
  ## [1.2.16](https://github.com/storm-software/storm-ops/compare/git-tools-v1.2.15...git-tools-v1.2.16) (2023-11-10)
2
12
 
3
13
 
@@ -9,14 +9,14 @@ try {
9
9
  'changedFiles="$(git diff-tree -r --name-only --no-commit-id $1 $2)" && node @storm-software/git-tools/scripts/package-version-warning.cjs $changedFiles'
10
10
  );
11
11
 
12
- const result = execSync(
13
- "command -v git-lfs >/dev/null 2>&1 || echo >&2 '\nThis repository is configured for Git LFS but 'git-lfs' was not found on your path. If you no longer wish to use Git LFS, remove this hook by deleting .git/hooks/post-checkout.\n' && exit(2) ",
14
- "utf8"
15
- );
16
- if (Number(result)) {
17
- console.error(`Git LFS error: ${result}`);
12
+ const result = execSync("git-lfs -v", "utf8");
13
+ if (result && Number.isInteger(Number.parseInt(result)) && Number(result)) {
14
+ console.error(
15
+ `This repository is configured for Git LFS but 'git-lfs' was not found on your path. If you no longer wish to use Git LFS, remove this hook by deleting .git/hooks/post-checkout.\nError: ${result}`
16
+ );
18
17
  process.exit(1);
19
18
  }
19
+
20
20
  execSync('git lfs post-checkout "$@"');
21
21
  } catch (e) {
22
22
  console.error(e);
@@ -5,14 +5,14 @@ const { execSync } = require("node:child_process");
5
5
  try {
6
6
  console.log("Running Storm post-commit hook...");
7
7
 
8
- const result = execSync(
9
- "command -v git-lfs >/dev/null 2>&1 || echo >&2 '\nThis repository is configured for Git LFS but 'git-lfs' was not found on your path. If you no longer wish to use Git LFS, remove this hook by deleting .git/hooks/post-commit.\n' && exit(2) ",
10
- "utf8"
11
- );
12
- if (Number(result)) {
13
- console.error(`Git LFS error: ${result}`);
8
+ const result = execSync("git-lfs -v", "utf8");
9
+ if (result && Number.isInteger(Number.parseInt(result)) && Number(result)) {
10
+ console.error(
11
+ `This repository is configured for Git LFS but 'git-lfs' was not found on your path. If you no longer wish to use Git LFS, remove this hook by deleting .git/hooks/post-commit.\nError: ${result}`
12
+ );
14
13
  process.exit(1);
15
14
  }
15
+
16
16
  execSync('git lfs post-commit "$@"');
17
17
  } catch (e) {
18
18
  console.error(e);
@@ -9,14 +9,14 @@ try {
9
9
  'changedFiles="$(git diff-tree -r --name-only --no-commit-id ORIG_HEAD HEAD)" && @storm-software/git-tools/scripts/package-version-warning.cjs $changedFiles'
10
10
  );
11
11
 
12
- const result = execSync(
13
- "command -v git-lfs >/dev/null 2>&1 || echo >&2 '\nThis repository is configured for Git LFS but 'git-lfs' was not found on your path. If you no longer wish to use Git LFS, remove this hook by deleting .git/hooks/post-merge.\n' && exit(2) ",
14
- "utf8"
15
- );
16
- if (Number(result)) {
17
- console.error(`Git LFS error: ${result}`);
12
+ const result = execSync("git-lfs -v", "utf8");
13
+ if (result && Number.isInteger(Number.parseInt(result)) && Number(result)) {
14
+ console.error(
15
+ `This repository is configured for Git LFS but 'git-lfs' was not found on your path. If you no longer wish to use Git LFS, remove this hook by deleting .git/hooks/post-merge.\nError: ${result}`
16
+ );
18
17
  process.exit(1);
19
18
  }
19
+
20
20
  execSync('git lfs post-merge "$@"');
21
21
  } catch (e) {
22
22
  console.error(e);
package/bin/pre-push.cjs CHANGED
@@ -7,14 +7,14 @@ try {
7
7
 
8
8
  execSync("node @storm-software/git-tools/scripts/check-lock-file.cjs");
9
9
 
10
- const result = execSync(
11
- "command -v git-lfs >/dev/null 2>&1 || echo >&2 '\nThis repository is configured for Git LFS but 'git-lfs' was not found on your path. If you no longer wish to use Git LFS, remove this hook by deleting .git/hooks/pre-push.\n' && exit(2) ",
12
- "utf8"
13
- );
14
- if (Number(result)) {
15
- console.error(`Git LFS error: ${result}`);
10
+ const result = execSync("git-lfs -v", "utf8");
11
+ if (result && Number.isInteger(Number.parseInt(result)) && Number(result)) {
12
+ console.error(
13
+ `This repository is configured for Git LFS but 'git-lfs' was not found on your path. If you no longer wish to use Git LFS, remove this hook by deleting .git/hooks/pre-push.\nError: ${result}`
14
+ );
16
15
  process.exit(1);
17
16
  }
17
+
18
18
  execSync('git lfs pre-push "$@"');
19
19
  } catch (e) {
20
20
  console.error(e);
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@storm-software/git-tools",
3
- "version": "1.2.17",
3
+ "version": "1.2.18",
4
4
  "private": false,
5
5
  "description": "⚡ A package containing various git tools used in Storm workspaces.",
6
6
  "keywords": [