Allow vernier gem to be used optionally, but don't require it#1172
Merged
castwide merged 4 commits intocastwide:v0.59from Mar 8, 2026
Merged
Allow vernier gem to be used optionally, but don't require it#1172castwide merged 4 commits intocastwide:v0.59from
castwide merged 4 commits intocastwide:v0.59from
Conversation
castwide
reviewed
Mar 8, 2026
| require 'vernier' | ||
| rescue LoadError | ||
| warn 'vernier gem not found. Install with: gem install vernier' | ||
| $stderr.puts 'vernier gem not found. Add this line to your .Gemfile and run `bundle install` to use this command:' |
Owner
There was a problem hiding this comment.
Since .Gemfile is nonstandard, this notification might be confusing to users who attempt to profile Solargraph in a different project where it doesn't exist. They'd need to have both solargraph and vermier in their Gemfile. If they're not using bundler, they should only need to have vermier installed at the system level.
Owner
There was a problem hiding this comment.
Another option that was previously discussed, we could hide the command.
Contributor
Author
There was a problem hiding this comment.
Yeah, I wasn't thinking about using this on anything but solargraph :)
Contributor
Author
There was a problem hiding this comment.
Rephrased a little more generically - folks who want to profile can figure this out :)
Closed
apiology
commented
Mar 8, 2026
Owner
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ensures solargraph can still be used on the Microsoft Windows platform