Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge solargraph-arc code into solargraph-rails #28

Closed
iftheshoefritz opened this issue Feb 27, 2022 · 14 comments
Closed

Merge solargraph-arc code into solargraph-rails #28

iftheshoefritz opened this issue Feb 27, 2022 · 14 comments
Assignees

Comments

@iftheshoefritz
Copy link
Owner

@alisnic will be joining solargraph-rails, and together we'll merge his excellent solargraph-arc project with solargraph-rails. I'm sure users will be especially excited for improvements like db/schema.rb parsing that this will bring.

Large parts of the solargraph-rails code will be replaced in the process, but the interface to solargraph (and hence the experience for users) will remain the same.

@julianrubisch
Copy link
Collaborator

That is some great news! Congrats!

@alisnic alisnic self-assigned this Feb 28, 2022
@alisnic
Copy link
Collaborator

alisnic commented Feb 28, 2022

@iftheshoefritz first small step is done, I merged model annotations support to solargraph-arc, now everything is ready to starting cooking the main PR to solargraph-rails

@alisnic alisnic mentioned this issue Mar 2, 2022
2 tasks
@alisnic
Copy link
Collaborator

alisnic commented Mar 14, 2022

I added note to solargraph-arc README and archived the repo 🚀

@emmohac
Copy link

emmohac commented Mar 17, 2022

Very excited to try this out.

You should also setup PayPal or something for donation because I couldn't find it.

@iftheshoefritz
Copy link
Owner Author

Comment in the Solargraph project's rails issue announcing 1.0.0.pre.1 is here

@alisnic
Copy link
Collaborator

alisnic commented Sep 5, 2022

@iftheshoefritz I think it's time to publish a relase. Thoughts?

@iftheshoefritz
Copy link
Owner Author

iftheshoefritz commented Sep 30, 2022

@alisnic agreed. Two thoughts:

  • builds are failing, let's fix those (broken :/ PR here: Fix github builds #42)
  • let's update the README to match the solargraph-arc functionality

@iftheshoefritz
Copy link
Owner Author

I'm away next week, but can do a release in the second week of October.

@alisnic
Copy link
Collaborator

alisnic commented Oct 3, 2022

Two thoughts

Makes sense, will do a PR to update README when I have some free time on my hands and will take a look at your PR. Regarding build errors though - last time I tried to fix it, I couldn't :( Something is off with Github CI env

@aelkoussy
Copy link

aelkoussy commented Oct 12, 2022

Hey, thanks for your great work! :)
So now the plugin depends on schema.rb not the annotate comments?
does that mean it won't work for people with structure.sql?

I tried the plugin but unfortunately I didn't get any completions (despite having the annotate comments)

Thanks for your efforts though!

Update: tried to create a schema.rb and still not working, is there anyway to check if the plugin is working or not please? 👀

@iftheshoefritz
Copy link
Owner Author

@aelkoussy right now v1.0.0.pre.1 uses schema.rb, but in theory still has support for annotate. There is not yet any support for structure.sql.

Previous versions only use annotate, which is what you get if you do not specify the pre release version.

Note that we will shortly move to 1.0.0, at which point you'll get the same support as 1.0.0.pre.1 by default.

If you're having a problem with annotate support (or any other issue), would you mind raising a new issue with more details?

@aelkoussy
Copy link

@iftheshoefritz Thanks for explaining, and sure, I will raise an issue for that with more details, thanks

@iftheshoefritz
Copy link
Owner Author

@alisnic after fixing the README I released 1.0.1 (I did a bad release and yanked 1.0.0 🤦‍♂️) with everything from main. We can work on the builds at our leisure without worrying about the release.

@alisnic
Copy link
Collaborator

alisnic commented Oct 17, 2022

Awesome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants