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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lock Linguist to Bundler 1.x #4383

Merged
merged 9 commits into from
Jan 16, 2019
Merged

Conversation

lildude
Copy link
Member

@lildude lildude commented Jan 15, 2019

As seen in #4340, tests are starting to fail because Travis has decided to switch to Bundler 2.x by default as of 3 Jan 2019 - https://docs.travis-ci.com/user/languages/ruby/#bundler-20 - and this is clashing with Licensed which expressly requires ~>1.10 due to it using some internal bundler functionality.

This PR locks us to bundler ~>1.10 too.

It took a few attempts for me to get Travis to actually do what I want it to do as the documented method wasn't working 馃槥 but I got there in the end.

Template removed as it's not relevant

Copy link
Contributor

@pchaigno pchaigno left a comment

Choose a reason for hiding this comment

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

LGTM!

Do we know when Licensed will move to Bundler 2.x? Or even if they plan to?

@lildude
Copy link
Member Author

lildude commented Jan 16, 2019

Do we know when Licensed will move to Bundler 2.x? Or even if they plan to?

Not sure when, but my colleague has it on his list of things to test and see if anything breaks. Once it updates to 2.x, we can revert this PR.

@lildude lildude merged commit f1be49d into master Jan 16, 2019
@lildude lildude deleted the lildude/specify-bundler-version branch January 16, 2019 16:17
@github-linguist github-linguist locked as resolved and limited conversation to collaborators Jun 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants