Copilot commented on code in PR #56290:
URL: https://github.com/apache/airflow/pull/56290#discussion_r2395203423
##########
go-sdk/bundle/bundlev1/bundlev1server/server.go:
##########
@@ -56,7 +57,7 @@ type ServerConfig struct{}
// Zero or more options to configure the server may also be passed. There are
no options yet, this is to allow
// future changes without breaking compatibility
func Serve(bundle bundlev1.BundleProvider, opts ...ServeOpt) error {
- shared.SetupViper("")
+ config.SetupViper("")
Review Comment:
The function call should be updated to handle the new return signature of
SetupViper which now returns (*viper.Viper, error). The returned viper instance
and potential error should be handled appropriately.
```suggestion
_, err := config.SetupViper("")
if err != nil {
return err
}
```
--
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]
