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

Remove Gemfile.lock for Ruby head #1292

Closed
wants to merge 1 commit into from

Conversation

andyw8
Copy link
Contributor

@andyw8 andyw8 commented Jan 10, 2024

This will prevent failures such as:

nokogiri-1.16.0-x86_64-darwin requires ruby version < 3.4.dev, >= 3.0, which is
incompatible with the current version, 3.4.0.dev

@andyw8 andyw8 requested a review from a team as a code owner January 10, 2024 16:00
@andyw8 andyw8 requested review from egiurleo and KaanOzkan January 10, 2024 16:00
@st0012
Copy link
Member

st0012 commented Jan 10, 2024

Do we need it for this project though? We don't have nokogiri and the head failure is mostly caused by differences between released Prism and CRuby head Prism.

@andyw8
Copy link
Contributor Author

andyw8 commented Jan 10, 2024

Yeah, I guess not.

@andyw8 andyw8 closed this Jan 10, 2024
@KaanOzkan
Copy link
Contributor

It could be useful in the future when a platform specific gem is added to Gemfile.

Also the current CI failure on ruby head seems legitimate https://github.com/Shopify/ruby-lsp/actions/runs/7477978717/job/20351904232#step:4:85

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.

3 participants