nastra commented on code in PR #3728:
URL: https://github.com/apache/parquet-java/pull/3728#discussion_r3954990707


##########
AGENTS.md:
##########


Review Comment:
   I've updated this and moved things into separate files, which are referenced 
from the main AGENTS.md.
   
   > Lastly, there's a few "MUST" rules in here (e.g. every assertThatThrownBy 
must contain an isInstanceOf chain + msg check). Do we have linter rules for 
these rather than just putting them in agent guidelines?
   
   The project doesn't have a linter that would be able to enforce this 
unfortunately



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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to