ISSUE-12: Id should be all caps

fraenky8 / tables-to-go

convert your database tables to structs easily

MIT License

233 stars 42 forks source link

ISSUE-12: Id should be all caps #16

Closed fraenky8 closed 5 years ago

fraenky8 commented 5 years ago

This PR fixes #12 and does the following:

Adds functionality to convert predefined words to upper-case (https://github.com/golang/go/wiki/CodeReviewComments#initialisms)
Adds a new command-line flag to disable this behaviour
Updates the documentation

kaihendry commented 5 years ago

I noticed it incorrectly capitalised mIDllename in my struct https://youtu.be/1mnMmMF3Dx0?t=862 :sweat_smile:

fraenky8 commented 5 years ago

Yes this is the downside of the implementation I did - because to follow idiomatic go I can't distinguish where the ID (or any other string) should be. If I only focus on start and beginning, this would make life easier of course - but not sure if it's then suitable for all.

You can disable this behaviour by passing -no-initialism to the tool.