Skip to content

Only display the screenshots failed message for runs that create a report (not full compile failures)#2703

Merged
riccardobl merged 2 commits intojMonkeyEngine:masterfrom
richardTingle:only-if-artifacts
Apr 14, 2026
Merged

Only display the screenshots failed message for runs that create a report (not full compile failures)#2703
riccardobl merged 2 commits intojMonkeyEngine:masterfrom
richardTingle:only-if-artifacts

Conversation

@richardTingle
Copy link
Copy Markdown
Member

Changes

  • The "Screenshot tests have failed" message will only be shown if a report is generated. Not (for example) if a compile failure means all steps fail
  • The "Screenshot tests have failed" message now includes a direct link to the screenshot report
  • The "Screenshot tests have failed" message updates itself (so new failing run will update the links, so they always point to the most recent failing screenshot run

Example message

image

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@riccardobl riccardobl merged commit e356496 into jMonkeyEngine:master Apr 14, 2026
19 checks passed
@yaRnMcDonuts yaRnMcDonuts added this to the v3.10.0 milestone Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants