revert wrong merge

matifali · matifali · commit a85a0aa6477d · 2023-06-16T13:32:16.000Z
diff --git a/.github/actions/setup-node/action.yaml b/.github/actions/setup-node/action.yaml
@@ -13,6 +13,3 @@ runs:
     - name: Install node_modules
       shell: bash
       run: ./scripts/yarn_install.sh
-    - name: Install depcheck
-      shell: bash
-      run : npm install -g depcheck
diff --git a/Makefile b/Makefile
@@ -408,27 +408,12 @@ lint: lint/shellcheck lint/go lint/ts lint/helm
 lint/ts:
 	cd site
 	yarn && yarn lint
-	depcheck || true
-	if [ -f unused.json ]; then
-		echo "Unused dependencies found:"
-		cat unused.json
-		echo "Please review and remove these dependencies manually."
-		exit 1
-	fi
 .PHONY: lint/ts
 
 lint/go:
 	./scripts/check_enterprise_imports.sh
 	go install github.com/golangci/golangci-lint/cmd/golangci-lint@v1.53.2
 	golangci-lint run
-	go mod tidy
-	git diff --exit-code -- go.mod go.sum
-	if [ $? -ne 0 ]; then
-		echo "Unused Go dependencies found:"
-		git diff go.mod go.sum
-		echo "Please review and remove these dependencies manually."
-		exit 1
-	fi
 .PHONY: lint/go
 
 # Use shfmt to determine the shell files, takes editorconfig into consideration.
diff --git a/dogfood/Dockerfile b/dogfood/Dockerfile
@@ -191,7 +191,7 @@ RUN apt-get update && \
 	google-chrome-stable microsoft-edge-beta && \
 	# Pre-install system dependencies that Playwright needs. npx doesn't work here
 	# for some reason. See https://github.com/microsoft/playwright-cli/issues/136
-	npm i -g playwright@1.19.1 && playwright install-deps depcheck
+	npm i -g playwright@1.19.1 && playwright install-deps
 
 # Ensure PostgreSQL binaries are in the users $PATH.
 RUN update-alternatives --install /usr/local/bin/initdb initdb /usr/lib/postgresql/13/bin/initdb 100 && \
