From 16008a64f6ab9309641f30b8a7c9a432f1649b9a Mon Sep 17 00:00:00 2001
From: "Justin M. Keyes" <justinkz@gmail.com>
Date: Tue, 26 Nov 2024 07:34:16 -0800
Subject: ci: check for deprecated util functions #3462

---
 .github/ci/run_sanitizer.sh     | 12 ++++++++++--
 .github/workflows/lint.yml      | 12 ++++++++++++
 .github/workflows/sanitizer.yml | 14 --------------
 3 files changed, 22 insertions(+), 16 deletions(-)
 delete mode 100644 .github/workflows/sanitizer.yml

(limited to '.github')

diff --git a/.github/ci/run_sanitizer.sh b/.github/ci/run_sanitizer.sh
index 8630de3f..1f148ba2 100644
--- a/.github/ci/run_sanitizer.sh
+++ b/.github/ci/run_sanitizer.sh
@@ -5,12 +5,20 @@ REF_BRANCH="$1"
 PR_BRANCH="$2"
 
 # checks for added lines that contain search pattern and prints them
-SEARCH_PATTERN="(path\.dirname|fn\.cwd)"
+SEARCH_PATTERN='(path\.dirname|fn\.cwd)'
 
 if git diff --pickaxe-all -U0 -G "${SEARCH_PATTERN}" "${REF_BRANCH}" "${PR_BRANCH}" -- '*.lua' | grep -Ev '(configs|utils)\.lua$' | grep -E "^\+.*${SEARCH_PATTERN}" ; then
   echo
   echo 'String "dirname" found. There is a high risk that this might contradict the directive:'
-  echo '"Do not add vim.fn.cwd or util.path.dirname in root_dir".'
+  echo '"Do not use vim.fn.cwd or util.path.dirname in root_dir".'
   echo "see: https://github.com/neovim/nvim-lspconfig/blob/master/CONTRIBUTING.md#adding-a-server-to-lspconfig."
   exit 1
 fi
+
+SEARCH_PATTERN='(util\.path\.dirname)'
+
+if git diff --pickaxe-all -U0 -G "${SEARCH_PATTERN}" "${REF_BRANCH}" "${PR_BRANCH}" -- '*.lua' | grep -Ev '\.lua$' | grep -E "^\+.*${SEARCH_PATTERN}" ; then
+  echo
+  echo 'Do not use deprecated util functions: '"${SEARCH_PATTERN}"
+  exit 1
+fi
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
index 7ee1244d..def839e9 100644
--- a/.github/workflows/lint.yml
+++ b/.github/workflows/lint.yml
@@ -6,6 +6,18 @@ on:
       - master
 
 jobs:
+  deprecated-functions:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+          ref: ${{ github.event.pull_request.head.sha }}
+      - run: |
+          if ! bash .github/ci/run_sanitizer.sh ${{ github.event.pull_request.base.sha }} ${{ github.event.pull_request.head.sha }}; then
+            exit 1
+          fi
+
   luacheck:
     runs-on: ubuntu-latest
     steps:
diff --git a/.github/workflows/sanitizer.yml b/.github/workflows/sanitizer.yml
deleted file mode 100644
index 89fd51e3..00000000
--- a/.github/workflows/sanitizer.yml
+++ /dev/null
@@ -1,14 +0,0 @@
-name: "Dirname Checker"
-on: [pull_request]
-jobs:
-  disallowed-root-checker:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-          ref: ${{ github.event.pull_request.head.sha }}
-      - run: |
-          if ! bash .github/ci/run_sanitizer.sh ${{ github.event.pull_request.base.sha }} ${{ github.event.pull_request.head.sha }}; then
-            exit 1
-          fi
-- 
cgit v1.2.3-70-g09d2