aws / aws/eks-anywhere

Rename sources to conform with snake case

Open
#6,460 0 comments 0 reactions 0 assignees View on GitHub
good first issue
Dominant language
Go
Stars
2.1k
Forks
328
Avg merge
1d 4h
Merged PRs (30d)
9

Description

**Context**
We have Go guidelines for developing source code in EKS-A that honor established conventions defined by the broader Go community. 1 guideline is to use `snake_case` for file names which aligns with both community and compiler expectations where the compiler has special handling for platform specific files and test files. For example, `foo_amd64.go` only compiles on amd64 platforms and `foo_test.go` only compiles with `go test`.

This issue tracks the packages not conforming to our standard.

**Tracking**
- [ ] [/cmd/eksctl-anywhere/cmd](../tree/main/cmd/eksctl-anywhere/cmd)
- [ ] [/cmd/eks-a-tool/cmd](../tree/main/cmd/eks-a-tool/cmd)
- [ ] [/internal/pkg/api](../tree/main/internal/pkg/api)
- [ ] [/internal/pkg/conformance](../tree/main/internal/pkg/conformance)
- [ ] [/internal/pkg/nutanix](../tree/main/internal/pkg/nutanix)
- [ ] [/internal/pkg/test](../tree/main/internal/pkg/test)
- [ ] [/internal/pkg/test/e2e](../tree/main/internal/pkg/test/e2e)
- [ ] [/pkg/api](../tree/main/pkg/api)
- [ ] [/pkg/aws](../tree/main/pkg/aws)
- [ ] [/pkg/cli](../tree/main/pkg/cli)
- [ ] [/pkg/clients/kubernetes](../tree/main/pkg/clients/kubernetes)
- [ ] [/pkg/clusterapi](../tree/main/pkg/clusterapi)
- [ ] [/pkg/clusterapi/machinehealthcheck](../tree/main/pkg/clusterapi/machinehealthcheck)
- [ ] [/pkg/clusterapi/yaml](../tree/main/pkg/clusterapi/yaml)
- [ ] [/pkg/clustermarshaller](../tree/main/pkg/clustermarshaller)
- [ ] [/pkg/config](../tree/main/pkg/config)
- [ ] [/pkg/controller/clusters](../tree/main/pkg/controller/clusters)
- [ ] [/pkg/controller/handlers](../tree/main/pkg/controller/handlers)
- [ ] [/pkg/curatedpackages](../tree/main/pkg/curatedpackages)
- [ ] [/pkg/executables](../tree/main/pkg/executables)
- [ ] [/pkg/features](../tree/main/pkg/features)
- [ ] [/pkg/filewriter](../tree/main/pkg/filewriter)
- [ ] [/pkg/git/factory](../tree/main/pkg/git/factory)
- [ ] [/pkg/git/gitclient](../tree/main/pkg/git/gitclient)
- [ ] [/pkg/govmomi](../tree/main/pkg/govmomi)
- [ ] [/pkg/networkutils](../tree/main/pkg/networkutils)
- [ ] [/pkg/providers/...](../tree/main/pkg/providers) (check all)
- [ ] [/pkg/registrymirror](../tree/main/pkg/registrymirror)
- [ ] [/pkg/templater](../tree/main/pkg/templater)
- [ ] [/pkg/types](../tree/main/pkg/types) (🤦🏻)
- [ ] [/pkg/validations/...](../tree/main/pkg/validations) (check all)

Contributor guide

Open the contributing guide

Research direction

Start by auditing the listed directories, including cmd/eksctl-anywhere/cmd, internal/pkg, pkg/providers, and pkg/validations, for Go source filenames that do not use snake_case. Rename the nonconforming files while preserving their contents, then verify that all tracked source files in the listed packages conform and the repository checks pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
developer-experience, tooling
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.