Skip to content

PLAT-3214 - Don't fail Test Report when there are no test results - #19

Merged
noorulkhan-n merged 1 commit into
mainfrom
work/nk/PLAT-3214-test-report-empty
Oct 16, 2026
Merged

noorulkhan-n merged 1 commit into
mainfrom
work/nk/PLAT-3214-test-report-empty

Conversation

@noorulkhan-n

@noorulkhan-n noorulkhan-n commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Tenant to Deploy to: N/A (no plugin changes)

Description

This PR fixes the Test Report check failing on any PR that doesn't change a plugin (for example #17, which only updated the README). When no plugin changes, no tests run, so Test Report has no results and fails.

I've added fail-on-empty: false to .github/workflows/test-report.yml so the check passes when there are no test results.

Moved from #18, which was opened from my fork.

Jira: PLAT-3214

PR Creation Checklist:

  • PR title includes a descriptive title
  • Added a label: breaking or non-breaking
  • Added a label: bug-fix, new-plugin, internal or enhancement
  • PR description written
  • Validate script run (N/A, no plugin changes)

Validate script output (for breaking change)

N/A

@noorulkhan-n noorulkhan-n added bug-fix Fix for bug non-breaking Changes that will not break the existing functionality of a plugin labels Oct 15, 2026
@noorulkhan-n
noorulkhan-n requested a review from JRoseDev October 15, 2026 14:06
@noorulkhan-n
noorulkhan-n merged commit 4590bd4 into main Oct 16, 2026
6 of 7 checks passed
@noorulkhan-n
noorulkhan-n deleted the work/nk/PLAT-3214-test-report-empty branch October 16, 2026 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix Fix for bug non-breaking Changes that will not break the existing functionality of a plugin

Development

Successfully merging this pull request may close these issues.

2 participants