Skip to content

ci(psalm): Fail taint analysis action when it had errors - #64622

Merged
provokateurin merged 2 commits into
masterfrom
ci/noid/code-analysis-failure
Sep 28, 2026
Merged

provokateurin merged 2 commits into
masterfrom
ci/noid/code-analysis-failure

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

Checklist

AI (if applicable)

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

@nickvergessen nickvergessen added this to the Nextcloud 36 milestone Sep 21, 2026
@nickvergessen nickvergessen self-assigned this Sep 21, 2026
@nickvergessen
nickvergessen requested a review from a team as a code owner September 21, 2026 11:53
@nickvergessen nickvergessen added 3. to review Waiting for reviews tests Related to tests labels Sep 21, 2026
@nickvergessen
nickvergessen requested review from CarlSchwan, come-nc, leftybournes and provokateurin and removed request for a team September 21, 2026 11:53
@nickvergessen
nickvergessen force-pushed the ci/noid/code-analysis-failure branch from 3db2b8d to 5712bd8 Compare September 21, 2026 11:58

@come-nc come-nc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Congrats, the job is red, you have to fix the errors now.

Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Joas Schilling <coding@schilljs.com>
Assisted-by: ClaudeCode:claude-opus-5.5
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
nickvergessen force-pushed the ci/noid/code-analysis-failure branch from 5712bd8 to 669bd90 Compare September 28, 2026 07:02
@provokateurin
provokateurin merged commit 606e1ae into master Sep 28, 2026
155 checks passed
@provokateurin
provokateurin deleted the ci/noid/code-analysis-failure branch September 28, 2026 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews AI assisted tests Related to tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants