Shellcheck: move excludes to action env

.github/workflows/lint.yml

@@ -1,10 +1,4 @@
-on:
+on: push
-  push:
-    branches:
-    - master
-  pull_request:
-    branches:
-    - master
 
 name: Lint
 
@@ -12,17 +6,17 @@ jobs:
   shellcheck:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v2
+      - uses: actions/checkout@v2
-    - name: shellcheck
+      - name: shellcheck
-      uses: ludeeus/action-shellcheck@0.5.0
+        uses: ludeeus/action-shellcheck@0.5.0
-      env:
+        env:
-        SHELLCHECK_OPTS: -e SC1091,SC2164,SC2034,SC1072,SC1073,SC1009
+          SHELLCHECK_OPTS: -e SC1091,SC2164,SC2034,SC1072,SC1073,SC1009
 
   shfmt:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v2
+      - uses: actions/checkout@v2
-    - name: shfmt
+      - name: shfmt
-      uses: bltavares/actions/shfmt@master
+        uses: bltavares/actions/shfmt@master
-      env:
+        env:
-        SHFMT_ARGS: -d
+          SHFMT_ARGS: -d

openvpn-install.sh

@@ -18,7 +18,6 @@ function tunAvailable() {
 function checkOS() {
 	if [[ -e /etc/debian_version ]]; then
 		OS="debian"
-		# shellcheck disable=SC1091
 		source /etc/os-release
 
 		if [[ $ID == "debian" || $ID == "raspbian" ]]; then
@@ -51,7 +50,6 @@ function checkOS() {
 			fi
 		fi
 	elif [[ -e /etc/system-release ]]; then
-		# shellcheck disable=SC1091
 		source /etc/os-release
 		if [[ $ID == "fedora" ]]; then
 			OS="fedora"
@@ -1206,7 +1204,6 @@ function removeUnbound() {
 
 function removeOpenVPN() {
 	echo ""
-	# shellcheck disable=SC2034
 	read -rp "Do you really want to remove OpenVPN? [y/n]: " -e -i n REMOVE
 	if [[ $REMOVE == 'y' ]]; then
 		# Get OpenVPN port from the configuration