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

Add tagliatelle linter #1906

Merged
merged 1 commit into from Apr 12, 2021
Merged

Add tagliatelle linter #1906

merged 1 commit into from Apr 12, 2021

Conversation

ldez
Copy link
Member

@ldez ldez commented Apr 11, 2021

tagliatelle is a linter that handles struct tags.

for now, it only handle the string casing.

https://github.com/ldez/tagliatelle

@ldez ldez added the linter: new Support new linter label Apr 11, 2021
@ldez ldez marked this pull request as draft April 11, 2021 22:41
@ldez ldez force-pushed the feat/tagliatelle branch 2 times, most recently from c20f89d to d7db4ae Compare April 11, 2021 23:20
@ldez ldez marked this pull request as ready for review April 12, 2021 00:48
Copy link
Member

@bombsimon bombsimon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice linter, lgtm! Would be nice with examples or links to each styling in the upstream repo but that's not really a part of this PR and more a nice to have. Might not be obvious to everyone what the difference between f.ex. camel and goCamel is.

.golangci.yml Outdated Show resolved Hide resolved
@ldez
Copy link
Member Author

ldez commented Apr 12, 2021

I improved the readme to add illustrations of the different string casing.

https://github.com/ldez/tagliatelle

If you are interested I created a comparison between some lib around this topic: https://github.com/ldez/samples-go-libs-string-cases

@ldez ldez merged commit 93df6f7 into golangci:master Apr 12, 2021
@ldez ldez deleted the feat/tagliatelle branch April 12, 2021 15:59
@ldez ldez added this to the v1.40 milestone Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
linter: new Support new linter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants