ci: fix slack notifications team alias ⭐

Signed-off-by: Salim Afiune Maya <[email protected]>
lacework · Jan 6, 2021 · ca51f92 · ca51f92
1 parent 78905bb
commit ca51f92
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -3,7 +3,7 @@ version: 2.1
 parameters:
   slack-mentions:
     type: string
-    default: 'allies'
+    default: 'S01JP5A3ACQ'
   only_for_branches:
     type: string
     default: 'master'

diff --git a/cli/cmd/package_manifest.go b/cli/cmd/package_manifest.go
@@ -56,6 +56,10 @@ func (c *cliState) GeneratePackageManifest() (*api.PackageManifest, error) {
 	defer func() {
 		c.Event.DurationMs = time.Since(start).Milliseconds()
 		if err == nil {
+			// if this function returns an error, most likely,
+			// the command will send a honeyvent with that error,
+			// therefore we should duplicate events and only send
+			// one here if there is NO error
 			c.SendHoneyvent()
 		}
 	}()