3pacccccc commented on PR #25928: URL: https://github.com/apache/pulsar/pull/25928#issuecomment-4620387874
> This is a typical issue from LLM. In my own project, you'd better update the `AGENTS.md` as well. > > BTW, this is a part of my own AGENTS.md in Java projects: > > ``` > ## Project-Specific Conventions > > These are conventions **not** enforced by tooling that you must follow: > > ... > - **Imports over FQCNs**: In Java code, prefer normal imports over fully qualified class names unless there is a real name collision that makes an FQCN necessary. > - **Avoid reflection for tests**: When tests need internal visibility, prefer package-private members first. If package access is not appropriate, use a minimal `@VisibleForTesting` accessor. Use reflection only when there is no reasonable alternative. > ``` added -- 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]
