Skip to content

Commit

Permalink
Merge pull request #1166 from dandi/reporting
Browse files Browse the repository at this point in the history
Corrected reporting function logic to complete group message variable
  • Loading branch information
yarikoptic committed Nov 29, 2022
2 parents 0439caf + d2d6836 commit 0ad1346
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions dandi/cli/cmd_validate.py
Original file line number Diff line number Diff line change
Expand Up @@ -199,14 +199,15 @@ def display_errors(
f"{errors[0]}: detected in {pluralize(len(purviews), 'purviews')}"
)
if len(set(messages)) == 1:
error_message += f" — {messages[0]}."
group_message += f" — {messages[0]}."
click.secho(group_message, fg=fg)
for purview, severity in zip(purviews, severities):
error_message = f" {purview}"
fg = _get_severity_color([severity])
click.secho(error_message, fg=fg)
else:
error_message += "."
group_message += "."
click.secho(group_message, fg=fg)
for purview, severity, message in zip(purviews, severities, messages):
error_message = f" {purview}{message}"
fg = _get_severity_color([severity])
Expand Down

0 comments on commit 0ad1346

Please sign in to comment.