start errors with first small letter

mantil-io · Nov 25, 2021 · b7cfb73 · b7cfb73
1 parent 20e27ea
commit b7cfb73
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 10 deletions.
diff --git a/cli/controller/controller.go b/cli/controller/controller.go
@@ -141,7 +141,7 @@ func newStoreWithStage(stageName string) (*domain.FileStore, *domain.Stage, erro
 	}
 	stage := fs.Stage(stageName)
 	if stage == nil {
-		return nil, nil, log.Wrapf("Stage %s not found", stageName)
+		return nil, nil, log.Wrapf("stage %s not found", stageName)
 	}
 	addDefer(func() { log.SetStage(fs, project, stage) })
 	return fs, stage, nil

diff --git a/cli/controller/new.go b/cli/controller/new.go
@@ -57,7 +57,7 @@ func createProject(name, from, moduleName string) error {
 		if errors.Is(err, git.ErrRepositoryNotFound) {
 			return log.Wrap(err, sourceNewUserError(repo))
 		}
-		return log.Wrap(err, "Could not initialize repository from source %s: %v", repo, err)
+		return log.Wrap(err, "could not initialize repository from source %s: %v", repo, err)
 
 	}
 	// delete LICENSE from template repositories
@@ -85,7 +85,7 @@ func repoURL(name, repo string) (string, error) {
 	if !isExternalRepo(repo) {
 		template := projectTemplate(repo)
 		if template == "" {
-			return "", log.Wrap(fmt.Errorf("invalid template %s", repo))
+			return "", log.Wrapf("invalid template %s", repo)
 
 		}
 		repo = TemplateRepos[template]

diff --git a/cli/controller/report.go b/cli/controller/report.go
@@ -76,7 +76,7 @@ func uploadLogs(days int, url string) error {
 	}
 	defer rsp.Body.Close()
 	if rsp.StatusCode != 200 {
-		return log.Wrapf("non-ok status received from logs upload: %s", rsp.Status)
+		return log.Wrapf("logs upload failed with status: %s", rsp.Status)
 	}
 	return nil
 }

diff --git a/cli/controller/setup.go b/cli/controller/setup.go
@@ -74,7 +74,7 @@ func NewSetup(a *SetupArgs) (*Setup, error) {
 
 func (c *Setup) Create(getPath func(string) (string, string)) error {
 	if !c.regionSupported() {
-		return log.Wrapf(`Mantil is currently not available in this region.
+		return log.Wrapf(`currently not available in this region
 Available regions are:
 	+ %s`, strings.Join(supportedAWSRegions, "\n\t+ "))
 	}
@@ -169,8 +169,7 @@ func (c *Setup) createSetupStack(acf domain.NodeFunctions, suffix string) error
 	}
 	stackWaiter := c.aws.CloudFormation().CreateStack(c.stackName, string(t), c.resourceTags)
 	if err := runStackProgress("Installing setup stack", types.ResourceStatusCreateComplete, stackWaiter); err != nil {
-		log.Error(err)
-		return log.Wrapf("Installing setup stack failed. No resources were created in your AWS account.")
+		return log.Wrap(err, "installing setup stack failed")
 	}
 	// https://github.com/aws-cloudformation/cloudformation-coverage-roadmap/issues/919
 	if err := c.aws.TagLogGroup(aws.LambdaLogGroup(c.lambdaName), c.resourceTags); err != nil {
@@ -182,11 +181,11 @@ func (c *Setup) createSetupStack(acf domain.NodeFunctions, suffix string) error
 func (c *Setup) Destroy() (bool, error) {
 	ws := c.store.Workspace()
 	if len(ws.Nodes) == 0 {
-		return false, log.Wrapf("Nothing to delete, there are no nodes installed in your workspace")
+		return false, log.Wrapf("nothing to delete, there are no nodes")
 	}
 	n := ws.Node(c.nodeName)
 	if n == nil {
-		return false, log.Wrapf("Node %s doesn't exist. For a complete list of available nodes run 'mantil aws ls'", c.nodeName)
+		return false, log.Wrapf("node %s doesn't exist\nFor a complete list of available nodes run 'mantil aws nodes'.", c.nodeName)
 	}
 	if !c.confirmDestroy(n) {
 		return false, nil

diff --git a/cli/controller/stage.go b/cli/controller/stage.go
@@ -276,7 +276,7 @@ func (s *Stage) List() error {
 func (s *Stage) Use() error {
 	stage := s.project.Stage(s.Stage)
 	if stage == nil {
-		return log.Wrapf("Stage %s not found", s.Stage)
+		return log.Wrapf("stage %s not found", s.Stage)
 	}
 	s.project.SetDefaultStage(s.Stage)
 	if err := s.store.Store(); err != nil {