From cb90f7296ecfe575d677ecf858bc7005adef78a6 Mon Sep 17 00:00:00 2001 From: Dylan Page Date: Wed, 7 Apr 2021 11:23:36 -0400 Subject: [PATCH] fix: status check comments to actually reflect their commands --- server/events/apply_command_runner.go | 2 +- server/events/approve_policies_command_runner.go | 2 +- server/events/plan_command_runner.go | 2 +- server/events/policy_check_command_runner.go | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/server/events/apply_command_runner.go b/server/events/apply_command_runner.go index 3088aa045f..9fd634d47a 100644 --- a/server/events/apply_command_runner.go +++ b/server/events/apply_command_runner.go @@ -132,7 +132,7 @@ func (a *ApplyCommandRunner) Run(ctx *CommandContext, cmd *CommentCommand) { ctx.Log.Info("determined there was no project to run apply in.") if !a.silenceVCSStatusNoProjects { // If there were no projects modified, we set successful commit statuses - // with 0/0 projects planned/policy_checked/applied successfully because some users require + // with 0/0 projects applied successfully because some users require // the Atlantis status to be passing for all pull requests. ctx.Log.Debug("setting VCS status to success with no projects found") if err := a.commitStatusUpdater.UpdateCombinedCount(baseRepo, pull, models.SuccessCommitStatus, models.ApplyCommand, 0, 0); err != nil { diff --git a/server/events/approve_policies_command_runner.go b/server/events/approve_policies_command_runner.go index 35854d6cec..c3c37dadb0 100644 --- a/server/events/approve_policies_command_runner.go +++ b/server/events/approve_policies_command_runner.go @@ -59,7 +59,7 @@ func (a *ApprovePoliciesCommandRunner) Run(ctx *CommandContext, cmd *CommentComm ctx.Log.Info("determined there was no project to run approve_policies in") if !a.silenceVCSStatusNoProjects { // If there were no projects modified, we set successful commit statuses - // with 0/0 projects planned/policy_checked/applied successfully because some users require + // with 0/0 projects approve_policies successfully because some users require // the Atlantis status to be passing for all pull requests. ctx.Log.Debug("setting VCS status to success with no projects found") if err := a.commitStatusUpdater.UpdateCombinedCount(ctx.Pull.BaseRepo, ctx.Pull, models.SuccessCommitStatus, models.ApprovePoliciesCommand, 0, 0); err != nil { diff --git a/server/events/plan_command_runner.go b/server/events/plan_command_runner.go index 55b27f77e9..7c65346811 100644 --- a/server/events/plan_command_runner.go +++ b/server/events/plan_command_runner.go @@ -157,7 +157,7 @@ func (p *PlanCommandRunner) run(ctx *CommandContext, cmd *CommentCommand) { ctx.Log.Info("determined there was no project to run plan in") if !p.silenceVCSStatusNoProjects { // If there were no projects modified, we set successful commit statuses - // with 0/0 projects planned/policy_checked/applied successfully because some users require + // with 0/0 projects planned successfully because some users require // the Atlantis status to be passing for all pull requests. ctx.Log.Debug("setting VCS status to success with no projects found") if err := p.commitStatusUpdater.UpdateCombinedCount(baseRepo, pull, models.SuccessCommitStatus, models.PlanCommand, 0, 0); err != nil { diff --git a/server/events/policy_check_command_runner.go b/server/events/policy_check_command_runner.go index a648c92cc1..246830fa92 100644 --- a/server/events/policy_check_command_runner.go +++ b/server/events/policy_check_command_runner.go @@ -36,7 +36,7 @@ func (p *PolicyCheckCommandRunner) Run(ctx *CommandContext, cmds []models.Projec ctx.Log.Info("no projects to run policy_check in") if !p.silenceVCSStatusNoProjects { // If there were no projects modified, we set successful commit statuses - // with 0/0 projects planned/policy_checked/applied successfully because some users require + // with 0/0 projects policy_checked successfully because some users require // the Atlantis status to be passing for all pull requests. ctx.Log.Debug("setting VCS status to success with no projects found") if err := p.commitStatusUpdater.UpdateCombinedCount(ctx.Pull.BaseRepo, ctx.Pull, models.SuccessCommitStatus, models.PolicyCheckCommand, 0, 0); err != nil {