alexandre-normand commented on issue #696: URL: https://github.com/apache/iceberg-go/issues/696#issuecomment-5295359703
> If you want to entirely remove the dependency on gocloud.dev from the go.mod then we'd need to make the integration tests an entirely separate go.mod and remove all references to io/gocloud/ from the *_test.go files Yeah, that's kind of required. The problem was never really binary size (for us, at least) but rather the fact that declaring a dependency on iceberg-go would force transitive upgrades to dependencies that have API breaking changes _and_ then force major lift to untangle different usages. This is more of a problem for monorepos than small projects but the impact is big enough that we have a rule that all libraries that have a dependency on gocloud.dev are required to vendor it (which is the reason we maintain a fork at the moment). -- 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]
