From fc3e8a356bd0d7c3c87b38290944db1a4fea70d4 Mon Sep 17 00:00:00 2001
From: Ian Fijolek <ian@iamthefij.com>
Date: Fri, 8 Jan 2021 16:45:05 -0500
Subject: [PATCH] Add drone pipeline

---
 .drone.yml  | 31 +++++++++++++++++++++++++++++++
 magefile.go | 34 ++++++++++++++++++++++------------
 2 files changed, 53 insertions(+), 12 deletions(-)
 create mode 100644 .drone.yml

diff --git a/.drone.yml b/.drone.yml
new file mode 100644
index 0000000..45ebf43
--- /dev/null
+++ b/.drone.yml
@@ -0,0 +1,31 @@
+---
+kind: pipeline
+name: test
+
+steps:
+  - name: check
+    image: iamthefij/drone-pre-commit:personal
+
+---
+kind: pipeline
+name: notify
+
+depends_on:
+  - test
+
+trigger:
+  status:
+    - failure
+
+steps:
+
+  - name: notify
+    image: drillster/drone-email
+    settings:
+      host:
+        from_secret: SMTP_HOST  # pragma: whitelist secret
+      username:
+        from_secret: SMTP_USER  # pragma: whitelist secret
+      password:
+        from_secret: SMTP_PASS  # pragma: whitelist secret
+      from: drone@iamthefij.com
diff --git a/magefile.go b/magefile.go
index 3780ca4..95c5998 100644
--- a/magefile.go
+++ b/magefile.go
@@ -29,13 +29,14 @@ func init() {
 	}
 }
 
-// Build workflow
+// Build workflow.
 func Build() error {
 	mg.Deps(cleanBuild)
 	fmt.Println("Building...")
+
 	err := sh.RunWith(info.Env(), "go", "build", "-o", buildDir+"/alfred-yubico-auth", ".")
 	if err != nil {
-		return err
+		return fmt.Errorf("error building binary %w", err)
 	}
 
 	globs := build.Globs(
@@ -49,54 +50,62 @@ func Build() error {
 	return build.SymlinkGlobs(buildDir, globs...)
 }
 
-// Run workflow
+// Run workflow.
 func Run() error {
 	mg.Deps(Build)
 	fmt.Println("Running...")
+
 	return sh.RunWith(info.Env(), buildDir+"/alfred-yubico-auth")
 }
 
-// Dist packages workflow for distribution
+// Dist packages workflow for distribution.
 func Dist() error {
 	mg.SerialDeps(Clean, Build)
+
 	p, err := build.Export(buildDir, distDir)
 	if err != nil {
 		return err
 	}
+
 	fmt.Printf("Exported %q\n", p)
+
 	return nil
 }
 
-// Install symlinked workflow to Alfred
+// Install symlinked workflow to Alfred.
 func Install() error {
 	mg.Deps(Build)
 	fmt.Printf("Installing (linking) %q to %q...\n", buildDir, info.InstallDir)
+
 	if err := sh.Rm(info.InstallDir); err != nil {
-		return err
+		return fmt.Errorf("error cleaning previously installed workflow: %w", err)
 	}
+
 	return build.Symlink(info.InstallDir, buildDir, true)
 }
 
-// InstallHooks will install pre-commit hooks
+// InstallHooks will install pre-commit hooks.
 func InstallHooks() error {
 	return sh.RunV("pre-commit", "install", "--overwrite", "--install-hooks")
 }
 
-// Check will run all pre-commit hooks
+// Check will run all pre-commit hooks.
 func Check() error {
 	return sh.RunV("pre-commit", "run", "--all-files")
 }
 
-// Clean build files
+// Clean build files.
 func Clean() error {
 	fmt.Println("Cleaning...")
 	mg.Deps(cleanBuild, cleanMage)
+
 	return nil
 }
 
-// DistClean build files and distribution files
+// DistClean build files and distribution files.
 func DistClean() error {
 	mg.Deps(Clean, cleanDist)
+
 	return nil
 }
 
@@ -107,14 +116,15 @@ func cleanDir(name string) error {
 
 	infos, err := ioutil.ReadDir(name)
 	if err != nil {
-		return err
+		return fmt.Errorf("cleanDir could not read folder: %w", err)
 	}
 
 	for _, fi := range infos {
 		if err := sh.Rm(filepath.Join(name, fi.Name())); err != nil {
-			return err
+			return fmt.Errorf("cleanDir could not remove file: %w", err)
 		}
 	}
+
 	return nil
 }