iftheshoefritz / solargraph-rails

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

Use schema.rb instead of annotate gem #24

Closed jhirn closed 2 years ago

jhirn commented 2 years ago

Hello. Great work here!

I was curious if the option to leverage schema.rb as opposed to the annotate gem has been considered as an option. I'm not a big fan of having a huge declaration that needs to be managed atop of all the model files and I believe it just pulls from the schema to do its work anyways. Not a knock on your approach here but you'd be open to the idea I'd take a stab at it and submit a PR.

iftheshoefritz commented 2 years ago

@jhirn If you have time to do a PR I would love to have it.

This is a duplicate of one of the earliest issues in the project :). Closing this for now, please post your updates in #10.

jhirn commented 2 years ago

Thank you! Didn't see the existing issue!