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.

Sign up for GitHub

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

Jump to bottom

paramgen generates wrong parameter name for an all-uppercase field #138

Closed
hariso opened this issue Nov 14, 2024 · 1 comment
Closed

paramgen generates wrong parameter name for an all-uppercase field #138

hariso opened this issue Nov 14, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@hariso
Copy link
Contributor

hariso commented Nov 14, 2024

Bug description

If a configuration struct contains a field with a name that's all upper-case, e.g.

FOO int

then paramgen generates the name foO for this field.

Steps to reproduce

See description.

Version

v0.5.0
@hariso hariso added the bug Something isn't working label Nov 14, 2024
This was referenced Nov 14, 2024
Merged
Merged
Open
@hariso
Copy link
Contributor Author

hariso commented Nov 15, 2024

The field name can be customzied with the json tag, so this isn't a big issue.

@hariso hariso closed this as completed Nov 15, 2024
@github-project-automation github-project-automation bot moved this from Triage to Done in Conduit Main Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
Conduit Main
Status: Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@hariso