Browse Source

fix(actions): Update psalm github action

Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
John Molakvoæ 1 năm trước cách đây
mục cha
commit
eab9ed6e09

+ 52 - 11
.github/workflows/psalm-github.yml

@@ -1,4 +1,4 @@
-name: Psalm show github
+name: Psalm
 
 on:
   pull_request:
@@ -8,21 +8,62 @@ on:
       - stable*
 
 jobs:
-  psalm:
-    name: Psalm
+  generate-report:
     runs-on: ubuntu-latest
+
     steps:
-      - name: Checkout code
+      - name: Checkout
         uses: actions/checkout@v3
         with:
-          submodules: recursive
+          submodules: true
+
+      - name: Set up php
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: '8.0'
+          extensions: ctype,curl,dom,fileinfo,gd,intl,json,mbstring,openssl,pdo_sqlite,posix,sqlite,xml,zip
+          coverage: none
+
+      - name: Composer install
+        run: composer i
+
       - name: Psalm
-        uses: docker://vimeo/psalm-github-actions:4.9.3
+        # Make sure we pass and still upload the report
         continue-on-error: true
+        run: composer run psalm -- --monochrome --no-progress --output-format=github --report=psalm.sarif
+
+      - name: Upload report to GitHub
+        if: always()
+        uses: github/codeql-action/upload-sarif@v2
         with:
-          composer_ignore_platform_reqs: false
-          report_file: results.sarif
-      - name: Upload Analysis results to GitHub
-        uses: github/codeql-action/upload-sarif@v1
+          sarif_file: psalm.sarif
+
+  generate-security-report:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+        with:
+          submodules: true
+
+      - name: Set up php
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: '8.0'
+          extensions: ctype,curl,dom,fileinfo,gd,intl,json,mbstring,openssl,pdo_sqlite,posix,sqlite,xml,zip
+          coverage: none
+
+      - name: Composer install
+        run: composer i
+
+      - name: Psalm
+        # Make sure we pass and still upload the report
+        continue-on-error: true
+        run: composer run psalm -- --monochrome --no-progress --output-format=github --taint-analysis --report=psalm.sarif
+
+      - name: Upload report to GitHub
+        if: always()
+        uses: github/codeql-action/upload-sarif@v2
         with:
-          sarif_file: results.sarif
+          sarif_file: psalm.sarif

+ 0 - 28
.github/workflows/psalm-security.yml

@@ -1,28 +0,0 @@
-name: Psalm Security Analysis
-
-on:
-  pull_request:
-  push:
-    branches:
-      - master
-      - stable*
-
-jobs:
-  psalm:
-    name: Psalm
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v3
-        with:
-          submodules: recursive
-      - name: Psalm
-        uses: docker://vimeo/psalm-github-actions:4.9.3
-        with:
-          security_analysis: true
-          composer_ignore_platform_reqs: false
-          report_file: results.sarif
-      - name: Upload Security Analysis results to GitHub
-        uses: github/codeql-action/upload-sarif@v1
-        with:
-          sarif_file: results.sarif

+ 3 - 1
.github/workflows/static-code-analysis.yml

@@ -1,4 +1,4 @@
-name: Static code analysis
+name: Psalm static code analysis
 
 on:
   pull_request:
@@ -6,6 +6,7 @@ on:
 jobs:
   static-code-analysis:
     runs-on: ubuntu-latest
+
     steps:
       - name: Checkout
         uses: actions/checkout@v3
@@ -34,6 +35,7 @@ jobs:
 
   static-code-analysis-ocp:
     runs-on: ubuntu-latest
+
     steps:
       - name: Checkout
         uses: actions/checkout@v3

+ 0 - 6
build/psalm-baseline.xml

@@ -578,15 +578,9 @@
     </UndefinedInterfaceMethod>
   </file>
   <file src="apps/dav/lib/Controller/InvitationResponseController.php">
-    <InvalidScalarArgument occurrences="1">
-      <code>$guests</code>
-    </InvalidScalarArgument>
     <UndefinedPropertyAssignment occurrences="1">
       <code>$vEvent-&gt;DTSTAMP</code>
     </UndefinedPropertyAssignment>
-    <UndefinedPropertyFetch occurrences="1">
-      <code>$vEvent-&gt;{'ATTENDEE'}</code>
-    </UndefinedPropertyFetch>
   </file>
   <file src="apps/dav/lib/DAV/CustomPropertiesBackend.php">
     <InvalidArgument occurrences="1">