gitedmond commented on PR #23691:
URL: https://github.com/apache/kafka/pull/23691#issuecomment-6270778891

   Hey @squah-confluent, I took a closer look at the change and what each test 
covers.
   I removed the 3 overlapping tests and 2 trailer subcases already covered by 
the second-pass checks, keeping the spacing assertion in the existing prose 
test.
   I strengthened the existing tests for fence boundaries, reference 
definitions, thematic breaks, and long identifiers. The only new test method 
checks whether the CLI edits the description: unchanged bodies must cause no 
edit, while CRLF normalization and prose reflow must submit the expected body.
   The updates are in 8680087 and e66e016. All 13 tests pass locally on Python 
3.14 and 3.12, and git diff --check passes. I also checked that the revised 
suite catches eight deliberately introduced regressions.


-- 
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