8000 minor #42594 Show nice CLI output in Psalm action (wouterj) · symfony/symfony@c22ba60 · GitHub
[go: up one dir, main page]
Skip to content

Commit c22ba60

Browse files
nicolas-grekasnicolas-grekas
committed
minor #42594 Show nice CLI output in Psalm action (wouterj)
This PR was merged into the 4.4 branch. Discussion ---------- Show nice CLI output in Psalm action | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | no | New feature? | no | Deprecations? | no | Tickets | - | License | MIT | Doc PR | - The GitHub format is cool, but rather useless on big PRs (especially when the Psalm comments are included in collapsed code blocks due to the size). It would be cool if GitHub improved this feature (e.g. by referencing the comments in the action output). Until then, I propose to merge this PR: It runs Psalm twice, once using the human readable CLI output and once using the GitHub output. The job duration is not impacted here, as Psalm caches its errors and thus only runs once. Commits ------- f75f758 Show nice CLI output in Psalm action
2 parents 83357b1 + f75f758 commit c22ba60

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

‎.github/workflows/psalm.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,4 +52,5 @@ jobs:
5252
5353
- name: Psalm
5454
run: |
55+
./vendor/bin/psalm.phar --no-progress
5556
./vendor/bin/psalm.phar --output-format=github --no-progress

0 commit comments

Comments
 (0)
0