Merge pull request #34 from tphoney/build_cleanup

(maint) cleaning build up with move to harness

Author: TP Honey

.gitignore

@@ -1,5 +1,3 @@
 /release/
 /drone-download*
-
 coverage.out
-.drone.yml

cmd/drone-download/main.go

@@ -3,14 +3,72 @@
 // Use of this source code is governed by an Apache 2.0 license that can be
 // found in the LICENSE file.
 
+// DO NOT MODIFY THIS FILE DIRECTLY
+
 package main
 
 import (
+	"os"
+
 	"github.com/drone-plugins/drone-download/plugin"
+	"github.com/drone-plugins/drone-plugin-lib/errors"
+	"github.com/drone-plugins/drone-plugin-lib/urfave"
+	"github.com/joho/godotenv"
 	"github.com/urfave/cli/v2"
 )
 
-// settingsFlags has the cli.Flags for the plugin.Settings.
+var version = "unknown"
+
+func main() {
+	settings := &plugin.Settings{}
+
+	if _, err := os.Stat("/run/drone/env"); err == nil {
+		_ = godotenv.Overload("/run/drone/env")
+	}
+
+	app := &cli.App{
+		Name:    "drone-download",
+		Usage:   "download a file",
+		Version: version,
+		Flags:   append(settingsFlags(settings), urfave.Flags()...),
+		Action:  run(settings),
+	}
+
+	if err := app.Run(os.Args); err != nil {
+		errors.HandleExit(err)
+	}
+}
+
+func run(settings *plugin.Settings) cli.ActionFunc {
+	return func(ctx *cli.Context) error {
+		urfave.LoggingFromContext(ctx)
+
+		plugin := plugin.New(
+			*settings,
+			urfave.PipelineFromContext(ctx),
+			urfave.NetworkFromContext(ctx),
+		)
+
+		if err := plugin.Validate(); err != nil {
+			if e, ok := err.(errors.ExitCoder); ok {
+				return e
+			}
+
+			return errors.ExitMessagef("validation failed: %w", err)
+		}
+
+		if err := plugin.Execute(); err != nil {
+			if e, ok := err.(errors.ExitCoder); ok {
+				return e
+			}
+
+			return errors.ExitMessagef("execution failed: %w", err)
+		}
+
+		return nil
+	}
+}
+
 func settingsFlags(settings *plugin.Settings) []cli.Flag {
 	return []cli.Flag{
 		&cli.StringFlag{

cmd/drone-download/config.go renamed to main.go

@@ -128,7 +128,7 @@ func (p *Plugin) Execute() error {
 
 	if exp != "" {
 		logrus.WithField("hash", alg).Info("Computing checksum")
-		target.Seek(0, 0)
+		_, _ = target.Seek(0, 0)
 
 		if _, err := io.Copy(h, target); err != nil {
 			defer os.Remove(target.Name())