-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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 exhaustivestruct linter #1411
Add exhaustivestruct linter #1411
Conversation
Hey, thank you for opening your first Pull Request ! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, could you please rebase on master and fix the conflicts?
Hey, @mbilski — we just merged your PR to
By joining the team, you’ll be able to label issues, review pull requests, and merge approved pull requests. Thanks again! |
@mbilski sorry to bring this up on a old thread but is there any way to specify the |
@sudo-suhas currently it's not possible. |
Add exhaustivestruct linter.
In my case, I use it to make sure that my SQL <-> Domain mappings always have all the fields initialized.
Definitely, it is a linter that you might want to use only for specific files where you want to make sure that you did not forget to add a field.