-
Notifications
You must be signed in to change notification settings - Fork 122
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
[tapioca add-on] Bump depend_on_ruby_lsp! version #2041
[tapioca add-on] Bump depend_on_ruby_lsp! version #2041
Conversation
lib/ruby_lsp/tapioca/addon.rb
Outdated
@@ -1,7 +1,7 @@ | |||
# typed: strict | |||
# frozen_string_literal: true | |||
|
|||
RubyLsp::Addon.depend_on_ruby_lsp!(">= 0.19.1", "< 0.20") | |||
RubyLsp::Addon.depend_on_ruby_lsp!(">= 0.20", "< 0.21") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to change it locally instead during development? I'm afraid of it degrading the internal testing since Ruby LSP isn't automatically bumped right away in my experience. I don't know the bumping logic and would like to learn also.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't notice the upper bound so ignore what I said above, it's better than failing to activate because Ruby LSP is too new.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 (and have updated to match the latest release).
lib/ruby_lsp/tapioca/addon.rb
Outdated
@@ -1,7 +1,7 @@ | |||
# typed: strict | |||
# frozen_string_literal: true | |||
|
|||
RubyLsp::Addon.depend_on_ruby_lsp!(">= 0.19.1", "< 0.20") | |||
RubyLsp::Addon.depend_on_ruby_lsp!(">= 0.20", "< 0.21") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't notice the upper bound so ignore what I said above, it's better than failing to activate because Ruby LSP is too new.
2a7cbae
to
02d4b7e
Compare
So that I can point to Ruby LSP's
main
during development.