Skip to content

ci: Simplify Psalm static analysis workflow, clarify push behavior, and cache Composer dependencies#59355

Draft
joshtrichards wants to merge 4 commits into
masterfrom
jtr/ci-refactor-static-code-analysis-simplify
Draft

ci: Simplify Psalm static analysis workflow, clarify push behavior, and cache Composer dependencies#59355
joshtrichards wants to merge 4 commits into
masterfrom
jtr/ci-refactor-static-code-analysis-simplify

Conversation

@joshtrichards
Copy link
Copy Markdown
Member

  • Resolves: #

Summary

Improve the Psalm static analysis workflow by reducing duplication and making its behavior easier to understand.

Changes:

  • clarify that the non-security Psalm jobs are pull-request-only, while keeping push runs for security analysis
  • refactor the PR analysis jobs into a matrix job
  • preserve the existing (recently added) changes gating and update the summary job accordingly
  • add Composer cache reuse to reduce repeated dependency download time
  • align PHP extension setup across the PR analysis variants

Notes:

  • push behavior remains in the same workflow because of existing dependencies elsewhere (i.e. repo-level security config)
  • the workflow path filters remain unchanged

TODO

  • ...

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Keep the combined workflow for now, but make it more obvious that the four regular Psalm jobs are PR-only while push is primarily for the security job. Behavior remains unchanged.

Signed-off-by: Josh <josh.t.richards@gmail.com>
…jobs

Signed-off-by: Josh <josh.t.richards@gmail.com>
Signed-off-by: Josh <josh.t.richards@gmail.com>
Signed-off-by: Josh <josh.t.richards@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants