-
Notifications
You must be signed in to change notification settings - Fork 277
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 HasUuids trait when Model has id uuid primary. #655
Conversation
…odel feat: add hasuuids trait to model
…odel test: add tests for hasuuids trait
Thanks for this. While the build is passing, you need to lint the code. You may do so by running There are some other refactors I would make, but if you can lint your code, I'll do the rest. |
Hey J Mac, I ran the linter. The code now conforms to the standard. |
thanks @rcrosbourne |
I cleaned up a few things, but there's more I'd like to fix. The functionality and test is there. So that's good enough to merge. I'll do the rest on master before merging. |
Addresses issue #646