From b3705aab06da58b098f05a55d1c433100d13db18 Mon Sep 17 00:00:00 2001 From: Oleg Bespalov Date: Fri, 13 Oct 2023 14:50:56 +0200 Subject: [PATCH] Apply suggestions from code review Co-authored-by: Ivan <2103732+codebien@users.noreply.github.com> --- cloudapi/config.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cloudapi/config.go b/cloudapi/config.go index 5e3da751684..ab8da31a636 100644 --- a/cloudapi/config.go +++ b/cloudapi/config.go @@ -321,7 +321,7 @@ func GetConsolidatedConfig( // we want to show a warning if the user is using the only old way of defining the config if cloudConfig == nil && external != nil { if _, ok := external[LegacyCloudConfigKey]; ok { - result.WarningMessage = "The options.ext.loadimpact option is deprecated, please use the a options.cloud instead" + result.WarningMessage = "The options.ext.loadimpact option is deprecated, please use options.cloud instead" } } @@ -379,7 +379,7 @@ func mergeFromCloudOptionAndExternal( // TODO: Fix this // We reuse cloud.Config for parsing options.cloud (or legacy loadimpact struct), but this probably shouldn't be // done, as the idea of options.ext is that they are extensible without touching k6. But in -// order for this to happen, we shouldn't actually marshall cloud.Config on top of it, because +// order for this to happen, we shouldn't actually marshal cloud.Config on top of it, because // it will be missing some fields that aren't actually mentioned in the struct. // So in order for use to copy the fields that we need for k6 cloud's api we unmarshal in // map[string]interface{} and copy what we need if it isn't set already