-
Notifications
You must be signed in to change notification settings - Fork 5
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
Struggling to get it working! #3
Comments
Try updating the gem first. Today I published version 0.17.1. It resolves an issue with infinite recursion loops in some workspace environments. Since you're getting messages about required paths in the console, I assume you're using it on an existing project. Sometimes it can take a while to process the maps on very large projects. Rails apps are especially susceptible to this. After the initial mapping, requests should become much faster. Make sure you're capitalizing If you still have problems, please let me know your OS, the versions of the Solargraph gem and the Atom package, and any errors you see in the console. |
Version 0.2.0 of the package fixes an issue with project directories that might resolve your problem. There's also a new version of the gem, so be sure to |
@castwide how to verify it's running? I'm running this in terminal -
|
@FunkyloverOne The Atom package is a little behind insofar as recent updates to the gem. Among other things, it still uses the old web API for integration instead of the language server protocol. I'm in the process of updating the package to use LSP. Plugins are still experimental and prone to bugs, so you might need to remove them from .solargraph.yml. You don't need to run Unfortunately, other problems are likely to persist until the package is updated. Related issue: #5 |
It occurred to me that you might be able to get it to work with an earlier version of the gem, so I tested a few of them and was able to get results with v0.17.4. Installing the gem:
Results in Atom: You still might encounter issues that have been resolved in the latest gem. I'll update #5 when the package is able to use it. |
@castwide this tool is awesome! Got it working today using the master branch. For anyone else struggling to get this running, I ran |
Thanks, @michielboekhoff! The master branch currently uses the language server protocol as mentioned in #5. I have a few more changes and tests to run before I publish the Atom package, but it should be close to ready. |
Version 0.3.0 with LSP support is published. |
@castwide For me version 0.3.0 still doesn't seem to work at all... I got my ruby installed via In atom console I got this error:
I guess I know how to fix it - you simply got to provide configuration parameter for solargraph executable path, just like |
Hey @castwide I see that you already have a So it's only needed to add such config to this package and pass it to SocketProvider |
@FunkyloverOne That should work, yes. I'll work on an update to atom-solargraph. I'm also in the process of an update to solargraph-utils that should help fix some other issues related to starting the server process. |
@castwide Any news on this? It'd be very cool to use solargraph together with chruby. PS: As a workaround, symlinking the solargraph binary into the PATH helps. In my case, I do something along the lines of |
@svoop The
I expect to publish the package in the next few days. |
@castwide I can confirm that I've finally succeeded! :D I had to start Atom from terminal though. Yet what we have now is a good start indeed! Thank you! |
Hey @castwide, here's what does the trick for reek and rubocop I believe: |
Thanks, @FunkyloverOne. That looks promising. Since the current version of atom-solargraph seems like a step in the right direction for most users, I'll still release it this week, but I'll refer to that thread for possible improvements. |
Version 0.4.0 is released with the commandPath configuration option. |
I've verified it's running in the console, received "Solargraph package activated."
It then prints that a lot of required paths aren't found.
However, I'm struggling to get it to autocomplete, I've tried running the example in the readme to get capitalise to show, but when I write "string." nothing appears, even after pressing control space
Any help would be appreciated!
The text was updated successfully, but these errors were encountered: