more idiomatic go

2 files changed, 7 insertions, 6 deletions

diff --git a/internal/config/server/server.go b/internal/config/server/server.go
index 109099e..7ecfac4 100644
--- a/internal/config/server/server.go
+++ b/internal/config/server/server.go
@@ -42,9 +42,10 @@ func New(configFile string) (ServerConfig, error) {
 	return conf, nil
 }
 
-func (conf ServerConfig) Partners() []string {
+func (conf ServerConfig) Partners() ([]string, error) {
 	if partners := strings.Split(conf.Partner, ","); partners[0] != "" {
-		return partners
+		return partners, nil
 	}
-	return []string{}
+
+	return []string{}, fmt.Errorf("no partners configured")
 }
diff --git a/internal/server/repository/repository.go b/internal/server/repository/repository.go
index b6a1edb..ffe4d37 100644
--- a/internal/server/repository/repository.go
+++ b/internal/server/repository/repository.go
@@ -187,9 +187,9 @@ func (r Repository) Merge(otherEnt types.Entry) error {
 func (r Repository) MergeRemotely(ctx context.Context) error {
 	var errs []error
 
-	partners := r.conf.Partners()
-	if len(partners) == 0 {
-		return fmt.Errorf("no parter for remote merge operation configured")
+	partners, err := r.conf.Partners()
+	if err != nil {
+		return err
 	}
 
 	for _, partner := range partners {