squah-confluent commented on PR #23691: URL: https://github.com/apache/kafka/pull/23691#issuecomment-6261232637
Thanks for the updates. Using AI tools is fine, but the PR author is expected to understand the code they submit, and that doesn't seem to be happening here. The latest round adds tests that duplicate existing ones: `test_preserves_timeline` repeats `test_preserves_list_markers`, `test_preserves_jira_fenced_code_reproduction` repeats `test_preserves_fenced_code_and_long_indented_lines`, and `test_normalizes_prose_spacing` repeats `test_wraps_plain_prose_at_72_columns`. Reviewing code the author hasn't checked themselves isn't a good use of reviewer time, and I'd rather not spend more of mine on it. Could you go through the change yourself and make sure you understand what each part does and why it's needed? I'm happy to pick the review back up once you have. -- 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]
