Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename files to the more commenly used (in Go projects) snake_case #141

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

svanharmelen
Copy link
Contributor

@svanharmelen svanharmelen commented Dec 10, 2024

In addition renamed a few identifiers to follow the initialisms guideline:
https://go.dev/wiki/CodeReviewComments#initialisms

@svanharmelen svanharmelen force-pushed the chore/file_names branch 2 times, most recently from 08f4d42 to 885be6d Compare December 10, 2024 08:53
@ccoVeille
Copy link
Contributor

ccoVeille commented Dec 10, 2024

There is a new revive rule for validating the filenames

Please add filename-format to the revive rules section of .golangci-lint (1.62.0 is needed)

Fixing the names is good, but making sure no more errors I'll be done is also important

@ccoVeille
Copy link
Contributor

There is also new revive rule for validating variables name

Please add var-naming to the revive rules section of .golangci-lint

@svanharmelen
Copy link
Contributor Author

svanharmelen commented Dec 12, 2024

Found two additional variables due to configuring the var-naming rule, so nice one @jorgerojas26 👍🏻

In addition one struct field is renamed to follow the initialisms
guideline: https://go.dev/wiki/CodeReviewComments#initialisms
@ccoVeille
Copy link
Contributor

Found two additional variables due to configuring the var-naming rule, so nice one @jorgerojas26 👍🏻

Nice one to me @ccoVeille , no ? 😂😅

@svanharmelen
Copy link
Contributor Author

Oops... Yes, nice one @ccoVeille 😉

@jorgerojas26
Copy link
Owner

Hahaha good work guys.

@jorgerojas26 jorgerojas26 merged commit 76bac19 into jorgerojas26:main Dec 12, 2024
2 checks passed
@svanharmelen svanharmelen deleted the chore/file_names branch December 13, 2024 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants