fix: fix linting errors

Author: damienbutt

.github/workflows/ci.yml

@@ -34,25 +34,11 @@ jobs:
 
       - name: Byte compile
         run: |
-          ./scripts/test-compile.sh
-
-      - name: Install package-lint
-        run: |
-          emacs -Q --batch \
-            --eval "(require 'package)" \
-            --eval "(add-to-list 'package-archives '(\"melpa\" . \"https://melpa.org/packages/\") t)" \
-            --eval "(package-initialize)" \
-            --eval "(package-refresh-contents)" \
-            --eval "(package-install 'package-lint)"
+          ./scripts/compile.sh
 
       - name: Run package-lint
         run: |
-          emacs -Q --batch \
-            --eval "(require 'package)" \
-            --eval "(add-to-list 'package-archives '(\"melpa\" . \"https://melpa.org/packages/\") t)" \
-            --eval "(package-initialize)" \
-            --eval "(require 'package-lint)" \
-            -f package-lint-batch-and-exit netlinx-mode.el netlinx-mode-font-lock.el
+          ./scripts/lint.sh
 
       - name: Check documentation
         run: |

netlinx-mode-font-lock.el

@@ -1,19 +1,15 @@
-;;; netlinx-mode-font-lock.el --- Font-lock settings for NetLinx mode -*- lexical-binding: t; -*-
+;;; netlinx-mode-font-lock.el --- Font-lock settings for NetLinx mode -*- lexical-binding: t; no-byte-compile: t; -*-
 
 ;; Copyright (C) 2024 Norgate AV
 
 ;; Author: Norgate AV
 ;; Maintainer: Norgate AV
-;; Version: 0.1.0
-;; Package-Requires: ((emacs "29.1"))
-;; Keywords: languages, netlinx, amx, harman
-;; URL: https://github.com/Norgate-AV/emacs-netlinx-mode
-;; SPDX-License-Identifier: MIT
 
 ;;; Commentary:
 
 ;; This file provides font-lock (syntax highlighting) settings for NetLinx mode
 ;; using tree-sitter. It defines custom faces and highlighting rules.
+;; This is an internal library for netlinx-mode.el and is not a standalone package.
 
 ;;; Code:
 

netlinx-mode.el

@@ -98,7 +98,7 @@ The file path is configured via `netlinx-mode-help-file'."
   (netlinx-mode--ensure-grammar)
 
   ;; Keybindings
-  (define-key netlinx-mode-map (kbd "C-c C-h") #'netlinx-open-help)
+  (define-key netlinx-mode-map (kbd "C-c C-d") #'netlinx-open-help)
 
   ;; Check if the NetLinx grammar is installed (t forces fresh check after installation)
   (if (treesit-ready-p 'netlinx t)

scripts/test-compile.sh scripts/compile.shscripts/test-compile.sh renamed to scripts/compile.sh

@@ -0,0 +1,39 @@
+#!/bin/bash
+
+set -e
+
+# Get the script directory and project root
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+PROJECT_ROOT="$(cd "$SCRIPT_DIR/.." && pwd)"
+
+# Files to lint
+FILES=("netlinx-mode.el")
+
+echo "Project root: $PROJECT_ROOT"
+echo "Files to lint: ${FILES[*]}"
+echo ""
+
+echo "Installing package-lint..."
+emacs -Q --batch \
+    --eval "(require 'package)" \
+    --eval "(add-to-list 'package-archives '(\"melpa\" . \"https://melpa.org/packages/\") t)" \
+    --eval "(package-initialize)" \
+    --eval "(package-refresh-contents)" \
+    --eval "(package-install 'package-lint)" || {
+    echo "Failed to install package-lint"
+    exit 1
+}
+
+echo ""
+echo "Running package-lint..."
+(cd "$PROJECT_ROOT" && emacs -Q --batch \
+    --eval "(require 'package)" \
+    --eval "(add-to-list 'package-archives '(\"melpa\" . \"https://melpa.org/packages/\") t)" \
+    --eval "(package-initialize)" \
+    --eval "(require 'package-lint)" \
+    -f package-lint-batch-and-exit "${FILES[@]}") || {
+    echo "Package lint failed"
+    exit 1
+}
+
+echo "Package lint succeeded for files: ${FILES[*]}"