Skip to content

XIVY-19238 Improve project validation summary reporting - #1159

Merged
ivy-edp merged 1 commit into
masterfrom
add-project-val-improvements$
Jul 23, 2026
Merged

XIVY-19238 Improve project validation summary reporting#1159
ivy-edp merged 1 commit into
masterfrom
add-project-val-improvements$

Conversation

@ivy-edp

@ivy-edp ivy-edp commented Jul 22, 2026

Copy link
Copy Markdown
Member
  • Log the summary block at the highest severity level found among the reported messages (error > warn > info), instead of always logging it at info level
  • List the skipped validators (from excludeValidators) as part of the summary, instead of logging them individually up front
grafik

@ivy-edp
ivy-edp requested a review from alexsuter July 22, 2026 13:58
@ivy-edp
ivy-edp requested review from ivy-lmu and ivy-rew as code owners July 22, 2026 13:58
@ivy-edp
ivy-edp requested a review from ivy-cst July 22, 2026 13:59

@ivy-cst ivy-cst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice ❤️
Before the summary, you could enter an info log to prevent Jenkins from reporting the summary and the last failure together.
You could log that the validation has finished, for example, and provide the duration or file count.

@ivy-edp
ivy-edp force-pushed the add-project-val-improvements$ branch 2 times, most recently from 8d364bc to 7214b71 Compare July 23, 2026 06:48
- Log the summary block at the highest severity level found among
  the reported messages (error > warn > info), instead of always
  logging it at info level
- List the skipped validators (from excludeValidators) as part of
  the summary, instead of logging them individually up front
@ivy-edp
ivy-edp force-pushed the add-project-val-improvements$ branch from 7214b71 to 03e2646 Compare July 23, 2026 06:49
@ivy-edp
ivy-edp merged commit 279b7b8 into master Jul 23, 2026
9 of 10 checks passed
@ivy-edp
ivy-edp deleted the add-project-val-improvements$ branch July 23, 2026 07:20
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.

2 participants