Move ShellCheck to Actions (#3036)

* Move ShellCheck to Actions

* Removed linting step from drone

PHP never caught really any bug in the future we should replace it with more advanced unit testing

Removed Publish  to apt step. Minor changes have been made an broke it.

Co-authored-by: Jaap Marcus <9754650+jaapmarcus@users.noreply.github.com>

.drone.yml

@@ -96,27 +96,6 @@ trigger:
   - refs/heads/main
   - refs/pull/*/head
 
----
-kind: pipeline
-type: docker
-name: Linting
-
-concurrency:
-  limit: 1
-
-steps:
-  - name: Shellcheck
-    image: koalaman/shellcheck-alpine:v0.8.0
-    commands:
-    - ./test/shellcheck.sh
-  - name: PHP 8.1
-    image: php:8.1-cli-bullseye
-    commands:
-      - ./test/check_php.sh ./web/
-
-trigger:
-      event: [ pull_request, push ]
-
 ---
 kind: pipeline
 type: docker
@@ -147,24 +126,7 @@ steps:
       target: /root/
       source:
         - ./hestia/*
-  - name: Publish
-    image: appleboy/drone-ssh
-    settings:
-        host:
-          from_secret: apt_server
-        user: root
-        key:
-          from_secret: ssh_key
-        port: 22
-        command_timeout: 2m
-        script:
-            - freight-add ./hestia/*.deb apt/bionic apt/focal apt/jammy apt/stretch apt/buster apt/bullseye
-            - freight-cache
-            - rm -fr ./hestia/
 
 trigger:
     event: [ promote]
 
----
-kind: signature
-hmac: f89a311b0f023c6ee92dbdb56bcd23ec663ef9242269a6b70c1c222bac4380ee

.github/workflows/lint.yml

@@ -0,0 +1,21 @@
+name: Lint
+
+on:
+  pull_request:
+  push:
+    branches:
+      - main
+
+jobs:
+  shellcheck:
+    name: ShellCheck
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v3
+
+      - name: Install ShellCheck
+        run: sudo apt-get install -y shellcheck
+
+      - name: Run ShellCheck
+        run: ./test/shellcheck.sh