Skip to content

fix: non snake case warning#3454

Merged
abonander merged 2 commits intolaunchbadge:mainfrom
joeydewaal:non-snake-case-warning
Aug 25, 2024
Merged

fix: non snake case warning#3454
abonander merged 2 commits intolaunchbadge:mainfrom
joeydewaal:non-snake-case-warning

Conversation

@joeydewaal
Copy link
Copy Markdown
Contributor

@joeydewaal joeydewaal commented Aug 23, 2024

Does your PR solve an issue?

This fixes a rust-analyzer warning when a query has a non snake-case name. It simply adds a #[allow(non_snake_case)] lint to the generated struct and variables.
Fixes: #3361

@joeydewaal joeydewaal changed the title Non snake case warning fix: non snake case warning Aug 23, 2024
@joeydewaal
Copy link
Copy Markdown
Contributor Author

This should also fix #3078

@abonander
Copy link
Copy Markdown
Collaborator

Do you still get lint warnings when accessing the generated fields?

@joeydewaal
Copy link
Copy Markdown
Contributor Author

Nope, no warnings

@abonander abonander merged commit 018ffe7 into launchbadge:main Aug 25, 2024
jrasanen pushed a commit to jrasanen/sqlx that referenced this pull request Oct 14, 2024
* fix: non-snake-case fields warning

* cargo fmt
jrasanen pushed a commit to jrasanen/sqlx that referenced this pull request Oct 14, 2024
* fix: non-snake-case fields warning

* cargo fmt
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.

macro uses wrong casing causing warnings in rust-analyzer

2 participants