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

Add Solargraph config, to enable its use in some LSP fluent editors #493

Merged
merged 1 commit into from
Dec 3, 2024

Conversation

faisal
Copy link
Contributor

@faisal faisal commented Jun 7, 2024

This adds sufficient Solargraph configuration to make it useful when editing in some LSP-compatible editors.

Check list:

@faisal faisal force-pushed the solargraph_config branch from 8f853ed to 712e59e Compare December 3, 2024 00:36
@faisal
Copy link
Contributor Author

faisal commented Dec 3, 2024

The ruby-head failure to be the same Cucumber issue seen in cucumber/cucumber-ruby#1769, expected to be fixed for Ruby 3.4 in the next Cucumber release.

Copy link
Collaborator

@etagwerker etagwerker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@faisal Looks good, thanks! 👍🏼

@etagwerker etagwerker merged commit 2744583 into whitesmith:main Dec 3, 2024
21 of 22 checks passed
@faisal faisal deleted the solargraph_config branch December 3, 2024 19:00
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 this pull request may close these issues.

2 participants