From 24344e963214f007473fbd2c106cd4d8269126ff Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sybren=20A=2E=20St=C3=BCvel?= Date: Tue, 28 Jun 2022 11:42:47 +0200 Subject: [PATCH] Cleanup: worker, simplify setting the manager URL The return value of `FileConfigWrangler.SetManagerURL()` was never used, so now the function doesn't return anything any more. --- internal/worker/config.go | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/internal/worker/config.go b/internal/worker/config.go index baf531b9..5c9e2e77 100644 --- a/internal/worker/config.go +++ b/internal/worker/config.go @@ -125,10 +125,8 @@ func (fcw *FileConfigWrangler) SaveCredentials(creds WorkerCredentials) error { // SetManagerURL overwrites the Manager URL in the cached configuration. // This is an in-memory change only, and will not be written to the config file. -// Returns a new copy of the WorkerConfig with the Manager URL updated. -func (fcw *FileConfigWrangler) SetManagerURL(managerURL string) WorkerConfig { +func (fcw *FileConfigWrangler) SetManagerURL(managerURL string) { fcw.wc.ManagerURL = managerURL - return *fcw.wc } // DefaultConfig returns a fairly sane default configuration.