search

laravel-shift / blueprint

A code generation tool for Laravel developers.
MIT License
2.82k stars 270 forks source link

Add HasUuids trait when Model has id uuid primary. #655

Closed rcrosbourne closed 9 months ago

rcrosbourne commented 9 months ago

Addresses issue #646

jasonmccreary commented 9 months ago

Thanks for this. While the build is passing, you need to lint the code. You may do so by running vendor/bin/pint, then submit your changes.

There are some other refactors I would make, but if you can lint your code, I'll do the rest.

rcrosbourne commented 9 months ago

Hey J Mac,

I ran the linter. The code now conforms to the standard.

brocard commented 9 months ago

thanks @rcrosbourne

jasonmccreary commented 9 months ago

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.