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

Fix Travis mime-types dependency issue #231

Conversation

gerrywastaken
Copy link
Contributor

Before this fix some of our Travis builds failed (during the gem install
stage) with:

Gem::InstallError: mime-types-data requires Ruby version >= 2.0.

Idea for the fix was found here:
sgruhier/foundation_rails_helper#116
Although we could not use the same method as I only want to lock
down the version of mime-type where necessary. Also trying to lock
the version to 2.6.2 would break our Rails-3.2 Appraisal path.

Before (on the main repo):
screen shot 2016-05-06 at 4 14 02 pm

After (on my fork):
screen shot 2016-05-06 at 4 14 39 pm

Note: The error seen in 22.4 was already failing it's just that it was hidden by the gem installation breaking.

Before this fix some of our Travis builds failed with:

    Gem::InstallError: mime-types-data requires Ruby version >= 2.0.

Idea for the fix was found here:
sgruhier/foundation_rails_helper#116
Although we could not use the same method as I only want to lock
down the version of mime-type where necessary. Also trying to lock
the version to 2.6.2 would break our Rails-3.2 Appraisal path.
@gerrywastaken gerrywastaken merged commit 28901dc into awesome-print:master May 6, 2016
@gerrywastaken
Copy link
Contributor Author

Note: The checks which are still failing were failing before this commit.

@gerrywastaken gerrywastaken deleted the fix-travis-mime-types-issue branch May 6, 2016 06:36
@gerrywastaken gerrywastaken restored the fix-travis-mime-types-issue branch May 6, 2016 06:36
@gerrywastaken
Copy link
Contributor Author

This fix is in version 1.7.0

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