Enable perlcritic checks in CI

Run perlcritic against all the perl git files. https://progress.opensuse.org/issues/138416 Signed-off-by: ybonatakis <[email protected]>
os-autoinst · Feb 12, 2024 · 66cb0a9 · 66cb0a9
1 parent 10683a4
commit 66cb0a9
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 0 deletions.
diff --git a/.github/workflows/check_critic.yml b/.github/workflows/check_critic.yml
@@ -0,0 +1,15 @@
+---
+name: perlcritic
+on: [push, pull_request]
+
+jobs:
+  perlcritic_checks:
+    runs-on: ubuntu-latest
+    container:
+      image: registry.opensuse.org/devel/openqa/containers/os-autoinst_dev
+    steps:
+      - uses: actions/checkout@v4
+      - name: Static analysis
+        run: |
+          git config --global --add safe.directory '*'
+          ./external/os-autoinst-common/tools/perlcritic --quiet $(git ls-files -- '*.p[ml]')
diff --git a/.perlcriticrc b/.perlcriticrc
@@ -0,0 +1 @@
+external/os-autoinst-common/.perlcriticrc
diff --git a/main.pm b/main.pm
@@ -2,6 +2,7 @@
 # SPDX-License-Identifier: GPL-2.0-or-later
 
 use strict;
+use warnings;
 use testapi;
 use autotest;
 

diff --git a/tests/boot.pm b/tests/boot.pm
@@ -3,6 +3,7 @@
 
 use base 'basetest';
 use strict;
+use warnings;
 use testapi;
 
 sub run {