search

iftheshoefritz / solargraph-rails

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

Support for associations #4

Closed iftheshoefritz closed 2 years ago

iftheshoefritz commented 3 years ago

If this gem claims to be for all of rails it should probably support associations. Annotate doesn't do associations so it would have to be implemented very differently.

julianrubisch commented 3 years ago

Yep but couldn’t the process be different? Contrary to normal attributes, associations are mapped in class code with the DSL

iftheshoefritz commented 3 years ago

It could be. I would then need to update the documentation to state that only part of the process uses annotate. Not such a big deal I guess.

iftheshoefritz commented 2 years ago

I haven't updated this issue in a while but if you install the gem using the instructions under Association Support in the README, the association DSL is supported. However, if you run solargraph bundle you'll lose a lot of intellisense because of an open solargraph issue:

https://github.com/castwide/solargraph/issues/473

iftheshoefritz commented 2 years ago

Closing this because associations have been in for a while. There are some problems but these can be raised in more specific issues.