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

RSPEC support #683

Open
reeganviljoen opened this issue Sep 8, 2023 · 4 comments · May be fixed by #716
Open

RSPEC support #683

reeganviljoen opened this issue Sep 8, 2023 · 4 comments · May be fixed by #716

Comments

@reeganviljoen
Copy link

I know their is support for Rails with solargraph through the solargraph-rails plugin , is their a solargraph-rspec plugin or built in support I can use for rspec

@castwide
Copy link
Owner

castwide commented Nov 6, 2023

There's an RSpec convention that currently ships with Solargraph, but it's far from complete. It's definitely a good candidate to become a separate extension.

@reeganviljoen
Copy link
Author

@castwide how would that candidacy process work for solargraph

@castwide
Copy link
Owner

castwide commented Dec 7, 2023

There's no official candidacy process. When I have time to work more on RSpec support, one of my first tasks would probably be to move it to its own extension. If someone else is interested in working on it, I'd be willing to assist.

@lekemula
Copy link

lekemula commented May 12, 2024

Hey @reeganviljoen @castwide I've got some good news for you here! After playing around for a bit I've been able to put some work and created an extension gem for RSpec, please check it out here: https://github.com/lekemula/solargraph-rspec.

It does not support all the possible use cases (as that would be humongous knowing the magic of RSpec) but hopefully, we'll get there one day!

Luckily after some iterations it ended up not needing any patches here, although I still think some of the initial changes there would be still beneficial. @castwide WDYT? I would also appreciate any feedback on the gem itself as I'm relatively a newbie in this domain. 🙏

Please give it a try and let me know how it works for you!

Cheers!

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

Successfully merging a pull request may close this issue.

3 participants