Skip to content

Merge pull request #25 from oreillymedia/CL-880-enable-aws-services #158

Merge pull request #25 from oreillymedia/CL-880-enable-aws-services

Merge pull request #25 from oreillymedia/CL-880-enable-aws-services #158

Triggered via push May 14, 2024 15:39
Status Success
Total duration 2m 24s
Artifacts

ci.yaml

on: push
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 1 warning
CI Build: cmd/log.go#L13
exported var ReasonSkip should have comment or be unexported
CI Build: cmd/log.go#L21
exported var ColorRegion should have comment or be unexported
CI Build: cmd/log.go#L27
comment on exported function Sorted should be of the form "Sorted ..."
CI Build: cmd/log.go#L43
exported function Log should have comment or be unexported
CI Build: cmd/nuke.go#L14
exported type Nuke should have comment or be unexported
CI Build: cmd/nuke.go#L24
exported function NewNuke should have comment or be unexported
CI Build: cmd/nuke.go#L33
exported method Nuke.Run should have comment or be unexported
CI Build: cmd/nuke.go#L37
error strings should not be capitalized or end with punctuation or a newline
CI Build: cmd/nuke.go#L128
error strings should not be capitalized or end with punctuation or a newline
CI Build: cmd/nuke.go#L147
exported method Nuke.Scan should have comment or be unexported
CI Build
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-go@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.