Copilot commented on code in PR #1990: URL: https://github.com/apache/pekko/pull/1990#discussion_r2248501419
########## actor/src/main/scala/org/apache/pekko/pattern/CircuitBreaker.scala: ########## @@ -777,7 +769,7 @@ class CircuitBreaker( /** * Internal state abstraction */ - private sealed trait State { + private[pattern] sealed trait State { Review Comment: Changing the visibility of the State trait from `private` to `private[pattern]` appears to be unrelated to the VarHandle migration and could be a breaking change for internal APIs. This change should be documented or moved to a separate commit. ```suggestion private sealed trait State { ``` -- 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: notifications-unsubscr...@pekko.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@pekko.apache.org For additional commands, e-mail: notifications-h...@pekko.apache.org