Skip to content

Commit 575faa7

Browse files
cultureamp-renovate[bot]liamstevensclaude
authored
chore(deps): update dependency golangci-lint to v2.7.2 (#83)
* chore(deps): update dependency golangci-lint to v2.7.2 * fix: update godoc comments to satisfy godoclint linter This fixes the go-lint check failure on PR #83 by updating three godoc comments to follow Go documentation conventions, where comments should start with the symbol name they're documenting. Changes made: - src/aws/ecs.go:15 - Updated EcsClientAPI comment to start with type name - src/aws/ecs.go:106 - Updated FindLogStreamFromTask comment to start with function name - src/aws/ssm.go:15 - Updated TaskRunnerConfiguration comment to start with type name These changes are required by the godoclint linter that is automatically enabled in golangci-lint v2.7.2 (the .golangci.yml config uses 'default: all'). Could not reproduce the lint failure locally as I have golangci-lint v1.60.1, but these changes align with standard Go documentation conventions and match the specific violations reported in the CI failure. Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]> --------- Co-authored-by: cultureamp-renovate[bot] <89962466+cultureamp-renovate[bot]@users.noreply.github.com> Co-authored-by: Liam Stevens <[email protected]> Co-authored-by: Claude <[email protected]>
1 parent 074ddc6 commit 575faa7

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

.github/workflows/go-checks.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ jobs:
3535
- name: Lint code
3636
uses: golangci/golangci-lint-action@v8
3737
with:
38-
version: v2.4.0
38+
version: v2.7.2
3939
working-directory: src
4040
args: "-v --timeout=2m"
4141

src/aws/ecs.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ import (
1212
"github.com/aws/aws-sdk-go-v2/service/ecs/types"
1313
)
1414

15-
// internal interface for ecs
15+
// EcsClientAPI is an internal interface for ecs
1616
type EcsClientAPI interface {
1717
RunTask(ctx context.Context, params *ecs.RunTaskInput, optFns ...func(*ecs.Options)) (*ecs.RunTaskOutput, error)
1818
DescribeTasks(ctx context.Context, params *ecs.DescribeTasksInput, optFns ...func(*ecs.Options)) (*ecs.DescribeTasksOutput, error)
@@ -103,7 +103,7 @@ func TaskIDFromArn(taskArn string) string {
103103
return parts[len(parts)-1]
104104
}
105105

106-
// Acquires LogStream details for given ECS Task
106+
// FindLogStreamFromTask acquires LogStream details for given ECS Task
107107
func FindLogStreamFromTask(ctx context.Context, ecsClientAPI EcsClientAPI, task types.Task) (LogDetails, error) {
108108
response, err := ecsClientAPI.DescribeTaskDefinition(ctx, &ecs.DescribeTaskDefinitionInput{
109109
TaskDefinition: task.TaskDefinitionArn,

src/aws/ssm.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ type ssmAPI interface {
1212
GetParameter(ctx context.Context, params *ssm.GetParameterInput, optFns ...func(*ssm.Options)) (*ssm.GetParameterOutput, error)
1313
}
1414

15-
// ECS Task Configuration
15+
// TaskRunnerConfiguration is ECS Task Configuration
1616
type TaskRunnerConfiguration struct {
1717
Cluster string `json:"cluster"`
1818
Command []string `json:"command" required:"false"`

0 commit comments

Comments
 (0)