voonhous opened a new pull request, #20422:
URL: https://github.com/apache/hudi/pull/20422

   ### Describe the issue this Pull Request addresses
   
   Follow-up to a review nit on #20421 (r4355622202): when the 
`pr_compliance.py` self-tests fail, can the author see why? The log did print, 
but behind one `::error::`, so the annotation showed only the first log line (a 
passing test) and the failures sat unlabelled in the step log.
   
   ### Summary and Changelog
   
   On failure, the "Test pr_compliance.py" step now prints the full log and 
raises one annotation naming the failed tests, e.g. `pr_compliance.py 
self-tests failed: bad title,bad body`. A passing run stays quiet, and the log 
still goes to the runner temp dir.
   
   This step only tests `pr_compliance.py` itself; a PR's own description 
problems are reported by the "Check PR description" step, which is unchanged.
   
   ### Impact
   
   CI output only.
   
   ### Risk Level
   
   none
   
   ### Documentation Update
   
   none
   
   ### Contributor's checklist
   
   - [x] Read through [contributor's 
guide](https://hudi.apache.org/contribute/how-to-contribute)
   - [x] Enough context is provided in the sections above
   - [ ] Adequate tests were added if applicable
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to