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

Upgrade CodeMirror to 5.48.2 #60

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

joergschiller
Copy link

Hi!

Is there anything more I can provide you with to consider merging this PR?

Kind regards,
Jörg

@joergschiller joergschiller changed the title Upgrade CodeMirror to 5.48.0 Upgrade CodeMirror to 5.48.2 Jul 22, 2019
@joergschiller
Copy link
Author

@fixlr just bumped it to 5.48.2

Using 1.4 doesn't work:

    Specified 'sqlite3' for database adapter, but the gem is not loaded. Add `gem 'sqlite3'` to your Gemfile (and ensure its version is at the minimum required by ActiveRecord). (Gem::LoadError)
Include newer ruby version/remove very old ones and relax version requirements. Kept 1.9 for now as it was handled  separately before.
Travis CI/RVM has no binaries.
Rails 4.2.8 is the first version which support Ruby 2.4.
bundler version incompatible
@joergschiller
Copy link
Author

joergschiller commented Jul 22, 2019

@fixlr I also changed the matrix of Travis CI builds to let them work with the published Ruby binaries (also added a dummy Rails 4.2).

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.

1 participant