search

mbarre / schemacrawler-additional-lints

Some additionnal lints for Schemacrawler
Other
12 stars 7 forks source link

[IDEA] : we have to add UUID type as well to this list #212

Open bheemreddy181-zz opened 5 years ago

bheemreddy181-zz commented 5 years ago

UUID is also a type where people are referring to for Primary Keys https://medium.com/@dylanwolff/uuid-primary-keys-with-postgres-and-rom-958492e223ea

https://github.com/mbarre/schemacrawler-additional-lints/blob/de95837f1c9bcebe2c59c57a733d6a9b30fd6230/src/main/java/io/github/mbarre/schemacrawler/utils/LintUtils.java#L85 do we want to make this method generic and change the lint name ?

bheemreddy181-zz commented 5 years ago

@adriens any thoughts , i can make a PR

adriens commented 5 years ago

Hi @bheemreddy181 , sorry for late answer I'm very busy and cannot spend as much time as required on this projetc. ..so does @mbarre .

Yes, I think this could be a good Idea to add UUID as a PK type. Agree @mbarre ?

adriens commented 3 years ago

See #267