search

db-migrate / pg

A postgresql driver for db-migrate.
Other
67 stars 51 forks source link

Add UUID type #67

Closed OussamaRomdhane closed 2 years ago

OussamaRomdhane commented 2 years ago

This depends on the uuid-ossp extension, so it can be questionable, but the purpose behind it is to eliminate the "false" warning messages.

Screenshot 2022-05-29 at 19 12 36
wzrdtales commented 2 years ago

well its a warning not an error. you don't need to fix every warning, its there to remind you, that you're operating off standard.

OussamaRomdhane commented 2 years ago

I still think this type is common enough for this to be slightly annoying

wzrdtales commented 2 years ago

but not in the standard and that is what matters. if you really would want to override, it would need to come from userspace instead and that is nothing implemented yet, so that would be more than just one line here. If you want and find it so annoying worth the effort, look to make it possible to pass custom types as configuration param.

otherwise, you will be like the other thousands, who don't care and use grep -v

OussamaRomdhane commented 2 years ago

Thank you so much that makes clear sense