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

RowToStructByName Snake Case Collision #2085

Merged
merged 4 commits into from
Jul 12, 2024
Merged

RowToStructByName Snake Case Collision #2085

merged 4 commits into from
Jul 12, 2024

Commits on Jul 12, 2024

  1. example test case that demonstrates snake case collision in db tags c… 

    …aused by rows.go 'fieldPosByName'
    @nolandseigler
    nolandseigler committed Jul 12, 2024
    Configuration menu
    Copy the full SHA
    7a35585 View commit details
    Browse the repository at this point in the history

  2. in rows.go 'fieldPosByName' use boolean to replace '_' and only execu… 

    …tion replacements when there are no db tags present
    @nolandseigler
    nolandseigler committed Jul 12, 2024
    Configuration menu
    Copy the full SHA
    7fceb64 View commit details
    Browse the repository at this point in the history

  3. formatting

    @nolandseigler
    nolandseigler committed Jul 12, 2024
    Configuration menu
    Copy the full SHA
    b25d092 View commit details
    Browse the repository at this point in the history

  4. use normalized equality or strict equality check in rows.go fieldPosB… 

    …yName
    @nolandseigler
    nolandseigler committed Jul 12, 2024
    Configuration menu
    Copy the full SHA
    71a8e53 View commit details
    Browse the repository at this point in the history