-
Notifications
You must be signed in to change notification settings - Fork 13
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
Clarify that this extension is not intended for general Ruby development #562
Comments
This extension is not intended for Shopify use only. Both the extension's description and the README accurately state what it is: an opinionated set of extensions with automatic configuration. Regarding the search results, I do not know how VS Code ranks extensions, but those assumptions don't seem to be accurate. The Ruby LSP also has Ruby in the name, also comes from the same verified organization, is configured to be the default extension for Ruby files in VS Code and yet is ranked below Solargraph and We could consider switching the name to something like |
Thanks for the reply @vinistock!
I said "naming", and the fact that a the single word "Ruby" extension is prioritized is consistent with other VSCode search results, which (I'd guess) prioritize exact matches.
I can't know, but I suspect it would.
I was directed to post this here by Shopify staff engineer Kevin Newton, who repeatedly asked me to check that the issues I was seeing weren't with this extension which is "meant for Shopify developers". The experience for newcomers at present is that the topmost un-deprecated extension for Ruby is either a) for general use; or b) NOT for general use, you silly person, why are you using that one? This seems unnecessarily confusing, and a name change would probably help. |
Sorry, that information was not correct. This is a generic extension pack that anyone can use, which comes with the set of extensions we usually work with at the company. Even if you don't use all extensions in the pack, you can still use it since they won't activate automatically. For example, the Sorbet extension will do absolutely nothing unless your project is fully onboarded to Sorbet. And RDBG is a debugger client, so it also does nothing unless you explicitly start debugging with it.
The naming and how VS Code ranks extensions is indeed unfortunate. We'll look into how we can change the display names. It might also be worth asking the VS Code team if they'd be open to make the default recommendation for a language always show up at the top if the user is querying for an exact language identifier. The Rust community has a similar issue, where |
I mean, respectfully, thanks for all the hard work and making it open source. But, I would love if you could state the conditions for which this extension is meant to work as the header of the description at the very least. Not just to say it is very, VERY opinionated. I see a few other extensions as well marked themselves as deprecated in favor of using this extension, but they seem at odds with their intention, they were more general purpose and I fear that a lot of useful tools disappeared that way.
|
I think there are a few misunderstandings here. Mainly, I believe you're confusing this extension pack with the Ruby LSP, which are not the same thing.
If you're talking about If other extensions (other than
We do mention it. There's an explanation of the rationale and possible workarounds in the extension's documentation.
We also mention this in a few places in our documentation. The extension troubleshooting guide has a section on Bundler related issues. And the server's design and roadmap document elaborates on the integration with Bundler.
This is definitely not the goal we're looking for. Can you please open a bug report with the workflow where it does not work? |
This issue is causing misunderstandings in the community and deviating from its original purpose. I created a separated issue for renaming the extension to clarify it's an extension pack #585, which we actually attempted but the marketplace doesn't accept it. We need to look into filing a request so that it can be renamed and prevent further confusion. I'm also adding a note to clarify that this extension doesn't implement any Ruby features #586. It's the extensions included in the pack that do. |
Through the magic of ✨algorithms✨, naming, and (presumably) being a verified org, this extension is currently the top non-deprecated result for "ruby" in the VSCode extensions shop. However, per Kevin Newton it is meant specifically for Shopify Rails development, not for general Ruby dev. There's probably nothing you can do to sink the extension below Ruby-LSP in the search results, but a clear notice in the description and README that this is intended for Shopify development would be a service to newcomers and eliminate some confusion.
The text was updated successfully, but these errors were encountered: