Skip to content

Refine error messages in batch processing#6142

Merged
evankanderson merged 1 commit intomindersec:mainfrom
evankanderson:refine-errors
Feb 28, 2026
Merged

Refine error messages in batch processing#6142
evankanderson merged 1 commit intomindersec:mainfrom
evankanderson:refine-errors

Conversation

@evankanderson
Copy link
Member

Summary

Add newlines after each "Skipping" message, and include the skip reason (to help with debugging).

Testing

Tested this while testing #6137 and custcodian/minder-rules-and-profiles#21

@evankanderson evankanderson requested a review from a team as a code owner February 26, 2026 22:50
JAORMX
JAORMX previously approved these changes Feb 27, 2026
Copy link
Member Author

@evankanderson evankanderson left a comment

Choose a reason for hiding this comment

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

Coveralls is still having an outage (they are very mad about it at https://status.coveralls.io/), but the lint run did catch a minor issue, so I had to push a fix.

@evankanderson evankanderson merged commit 0f98d98 into mindersec:main Feb 28, 2026
24 checks passed
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