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.

Sign up for GitHub

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

Jump to bottom

Upgrade Ruby LSP requirement to v0.13.0 #208

Merged
merged 5 commits into from
Dec 4, 2023
Merged

Upgrade Ruby LSP requirement to v0.13.0 #208

merged 5 commits into from
Dec 4, 2023

Conversation

vinistock
Copy link
Member

This PR does three things split by commit:

  1. Upgrade the Ruby LSP requirement to the latest one
  2. Fix the related breaking changes
  3. Start using the code lens test hierarchy, which is basically just a copy of Make test CodeLens request aware of test hierarchy ruby-lsp#1214

@vinistock vinistock self-assigned this Nov 30, 2023
@vinistock vinistock requested a review from a team as a code owner November 30, 2023 18:11
@vinistock vinistock requested review from andyw8 and Morriar November 30, 2023 18:11
@vinistock vinistock force-pushed the vs/upgrade_ruby_lsp_requirement branch from 17159bd to 0385945 Compare November 30, 2023 18:19
@vinistock vinistock requested a review from st0012 November 30, 2023 18:23
@vinistock vinistock merged commit aaf28eb into main Dec 4, 2023
20 of 26 checks passed
@vinistock vinistock deleted the vs/upgrade_ruby_lsp_requirement branch December 4, 2023 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@st0012 st0012 st0012 approved these changes

@andyw8 andyw8 Awaiting requested review from andyw8 andyw8 was automatically assigned from Shopify/ruby-dev-exp

@Morriar Morriar Awaiting requested review from Morriar Morriar was automatically assigned from Shopify/ruby-dev-exp

Assignees

@vinistock vinistock

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vinistock @st0012