iftheshoefritz / solargraph-rails

Solargraph plugin to add awareness of Rails-specific code
MIT License
321 stars 29 forks source link

Added support for postgres citext #35

Closed gracepetryk closed 2 years ago

gracepetryk commented 2 years ago

My company uses citext to store a bunch of case insensitive columns, this change should mark in any columns in the annotation with that type as Strings.

gracepetryk commented 2 years ago

docs for the type if you're not familiar: https://www.postgresql.org/docs/current/citext.html

alisnic commented 2 years ago

thanks for PR! can also please add an example and assertion to https://github.com/iftheshoefritz/solargraph-rails/blob/main/spec/solargraph-rails/schema_spec.rb?

gracepetryk commented 2 years ago

done, lmk if there's anything else you need me to do

alisnic commented 2 years ago

Awesome! thank you 🙏