Skip to content

Commit 308c17f

Browse files
shogo82148bmoffatt
andauthored
fix: staticcheck warnings in events/codebuild.go (#323)
Co-authored-by: Bryan Moffatt <bmoffatt@users.noreply.github.com>
1 parent b0cb7b2 commit 308c17f

File tree

2 files changed

+18
-27
lines changed

2 files changed

+18
-27
lines changed

.golangci.yml

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -17,12 +17,3 @@ run:
1717
skip-files:
1818
# These were code-generated, and cannot be changed without breaking RPC compatibility.
1919
- lambda/messages/*.go
20-
21-
# FIXME
22-
# events/codebuild.go:18:2: SA9004: only the first constant in this group has an explicit type (staticcheck)
23-
# CodeBuildPhaseStatusFailed CodeBuildPhaseStatus = "FAILED"
24-
# ^
25-
# events/codebuild.go:31:2: SA9004: only the first constant in this group has an explicit type (staticcheck)
26-
# CodeBuildPhaseTypeSubmitted CodeBuildPhaseType = "SUBMITTED"
27-
# ^
28-
- events/codebuild.go

events/codebuild.go

Lines changed: 18 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -16,29 +16,29 @@ type CodeBuildPhaseStatus string
1616

1717
const (
1818
CodeBuildPhaseStatusFailed CodeBuildPhaseStatus = "FAILED"
19-
CodeBuildPhaseStatusFault = "FAULT"
20-
CodeBuildPhaseStatusInProgress = "IN_PROGRESS"
21-
CodeBuildPhaseStatusQueued = "QUEUED"
22-
CodeBuildPhaseStatusStopped = "STOPPED"
23-
CodeBuildPhaseStatusSucceeded = "SUCCEEDED"
24-
CodeBuildPhaseStatusTimedOut = "TIMED_OUT"
19+
CodeBuildPhaseStatusFault CodeBuildPhaseStatus = "FAULT"
20+
CodeBuildPhaseStatusInProgress CodeBuildPhaseStatus = "IN_PROGRESS"
21+
CodeBuildPhaseStatusQueued CodeBuildPhaseStatus = "QUEUED"
22+
CodeBuildPhaseStatusStopped CodeBuildPhaseStatus = "STOPPED"
23+
CodeBuildPhaseStatusSucceeded CodeBuildPhaseStatus = "SUCCEEDED"
24+
CodeBuildPhaseStatusTimedOut CodeBuildPhaseStatus = "TIMED_OUT"
2525
)
2626

2727
// CodeBuildPhaseType represents the type of the code build phase (i.e. submitted, install)
2828
type CodeBuildPhaseType string
2929

3030
const (
3131
CodeBuildPhaseTypeSubmitted CodeBuildPhaseType = "SUBMITTED"
32-
CodeBuildPhaseTypeQueued = "QUEUED"
33-
CodeBuildPhaseTypeProvisioning = "PROVISIONING"
34-
CodeBuildPhaseTypeDownloadSource = "DOWNLOAD_SOURCE"
35-
CodeBuildPhaseTypeInstall = "INSTALL"
36-
CodeBuildPhaseTypePreBuild = "PRE_BUILD"
37-
CodeBuildPhaseTypeBuild = "BUILD"
38-
CodeBuildPhaseTypePostBuild = "POST_BUILD"
39-
CodeBuildPhaseTypeUploadArtifacts = "UPLOAD_ARTIFACTS"
40-
CodeBuildPhaseTypeFinalizing = "FINALIZING"
41-
CodeBuildPhaseTypeCompleted = "COMPLETED"
32+
CodeBuildPhaseTypeQueued CodeBuildPhaseType = "QUEUED"
33+
CodeBuildPhaseTypeProvisioning CodeBuildPhaseType = "PROVISIONING"
34+
CodeBuildPhaseTypeDownloadSource CodeBuildPhaseType = "DOWNLOAD_SOURCE"
35+
CodeBuildPhaseTypeInstall CodeBuildPhaseType = "INSTALL"
36+
CodeBuildPhaseTypePreBuild CodeBuildPhaseType = "PRE_BUILD"
37+
CodeBuildPhaseTypeBuild CodeBuildPhaseType = "BUILD"
38+
CodeBuildPhaseTypePostBuild CodeBuildPhaseType = "POST_BUILD"
39+
CodeBuildPhaseTypeUploadArtifacts CodeBuildPhaseType = "UPLOAD_ARTIFACTS"
40+
CodeBuildPhaseTypeFinalizing CodeBuildPhaseType = "FINALIZING"
41+
CodeBuildPhaseTypeCompleted CodeBuildPhaseType = "COMPLETED"
4242
)
4343

4444
// CodeBuildEvent is documented at:
@@ -81,12 +81,12 @@ type CodeBuildEventDetail struct {
8181
ProjectName string `json:"project-name"`
8282
BuildID string `json:"build-id"`
8383
AdditionalInformation CodeBuildEventAdditionalInformation `json:"additional-information"`
84-
CurrentPhase CodeBuildPhaseStatus `json:"current-phase"`
84+
CurrentPhase CodeBuildPhaseType `json:"current-phase"`
8585
CurrentPhaseContext string `json:"current-phase-context"`
8686
Version string `json:"version"`
8787

8888
CompletedPhaseStatus CodeBuildPhaseStatus `json:"completed-phase-status"`
89-
CompletedPhase CodeBuildPhaseStatus `json:"completed-phase"`
89+
CompletedPhase CodeBuildPhaseType `json:"completed-phase"`
9090
CompletedPhaseContext string `json:"completed-phase-context"`
9191
CompletedPhaseDuration DurationSeconds `json:"completed-phase-duration-seconds"`
9292
CompletedPhaseStart CodeBuildTime `json:"completed-phase-start"`

0 commit comments

Comments
 (0)