pvary opened a new pull request, #18293: URL: https://github.com/apache/iceberg/pull/18293
State which methods are expected to carry Javadoc. The section only said to default to no Javadoc, which gave no signal that public API is different from an internal helper. Replace the "user-visible regression" wording in the testing section. Nearly all Iceberg code is internal but observable through the public API, so that phrase did not describe the intended bar. The rule is about what a failing test tells the reader, so state that instead. -- **AI Disclosure** - Model: Claude Opus 5 - Platform/Tool: GitHub Copilot CLI - Human Oversight: fully reviewed - Prompt Summary: Add a Javadoc visibility rule to AGENTS.md and reword the "user-visible regression" testing bullet after a reviewer found it unclear. Generated-by: GitHub Copilot CLI (Claude Opus 5) -- 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]
