hooks for shellcheck, json and yaml check

INTO-CPS-Association · Dec 1, 2023 · 2b213fe · 2b213fe
1 parent 0a057c2
commit 2b213fe
Show file tree

Hide file tree

Showing 6 changed files with 29 additions and 7 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -66,3 +66,21 @@ repos:
       - id: markdownlint
         files: '^(client|servers/execution/runner|servers/lib)/.*\.md$'
         stages: [pre-commit]
+
+  - repo: https://github.com/syntaqx/git-hooks
+    rev: v0.0.18
+    hooks:
+      - id: shellcheck
+        files: '^(docs|deploy|script|ssl)/.*'
+        stages: [pre-commit]
+
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v4.5.0
+    hooks:
+      - id: check-json
+        files: '^(docs|deploy|script|ssl)/.*'
+        stages: [pre-commit]
+      - id: check-yaml
+        files: '^(docs|deploy|script|ssl)/.*'
+        stages: [pre-commit]
+
diff --git a/client/src/components/tab/TabComponent.tsx b/client/src/components/tab/TabComponent.tsx
@@ -24,7 +24,7 @@ function renderScopeTabList(scope: TabData[][], subIndex: number): JSX.Element {
 
 function renderScopeTabPanels(
   scope: TabData[][],
-  subIndex: number
+  subIndex: number,
 ): JSX.Element {
   return (
     <>

diff --git a/client/src/route/library/Library.tsx b/client/src/route/library/Library.tsx
@@ -32,7 +32,7 @@ export function createCombinedTabs() {
           />
         </>
       ),
-    }))
+    })),
   );
 }
 

diff --git a/client/test/unitTests/Components/TabComponent.test.tsx b/client/test/unitTests/Components/TabComponent.test.tsx
@@ -11,7 +11,7 @@ describe('TabComponent', () => {
 
   test('renders an empty tab', () => {
     const { getByText } = render(
-      <TabComponent assetType={assetTypeTabs} scope={scopeTabs} />
+      <TabComponent assetType={assetTypeTabs} scope={scopeTabs} />,
     );
     const emptyTab = getByText('Functions');
     expect(emptyTab).toBeInTheDocument();
@@ -60,10 +60,10 @@ describe('TabComponent', () => {
     }
 
     expect(
-      screen.queryByText(assetTypeTabs[0].body.props.children)
+      screen.queryByText(assetTypeTabs[0].body.props.children),
     ).not.toBeInTheDocument();
     expect(
-      screen.queryByText(assetTypeTabs[1].body.props.children)
+      screen.queryByText(assetTypeTabs[1].body.props.children),
     ).not.toBeInTheDocument();
   });
 

diff --git a/script/docs.sh b/script/docs.sh
@@ -13,7 +13,7 @@ COMMIT_HASH=$(git rev-parse --short HEAD)
 export COMMIT_HASH
 export MKDOCS_ENABLE_PDF_EXPORT=1 
 
-echo ${VERSION}
+echo "${VERSION}"
 if [ -d site ]
 then
   rm -rf site

diff --git a/script/env.sh b/script/env.sh
@@ -22,6 +22,10 @@ sudo -H pip3 install qrcode
 sudo apt-get install -y rubygems
 sudo gem install mdl
 
+# Install shellcheck
+sudo apt-get install -y shellcheck
+
 # Install madge for generating dependency graphs of typescript projects
 sudo apt-get install -y graphviz
-sudo npm install -g madge
+sudo npm install -g madge
+
-Original file line number
+Diff line change
@@ Expand Up / @@ -32,7 +32,7 @@ export function createCombinedTabs() { @@
               />
             </>
           ),
-        }))
+        })),
       );
     }
@@ Expand Down @@