nssalian commented on code in PR #3949: URL: https://github.com/apache/iceberg-python/pull/3949#discussion_r3994234449
########## AGENTS.md: ########## @@ -90,6 +90,12 @@ lives under `pyiceberg/`, organized by concern rather than by engine: - The Apache License header is required on every new source file (enforced by `./dev/check-license`). - Run `make lint` and `make test` before pushing; CI runs both plus the lockfile check. +## GitHub Write Approval + +- **Never** modify GitHub state without explicit user approval for that specific action in the current session; treat GitHub as read-only by default. Review Comment: An agent can bypass this. Something like stop hooks have helped if some git / other commands are attempted to being run. So I'm not sure of the value of adding it here versus having users keep their local setup guardrails. -- 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]
