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

Update carrierwave to 1.0, which supports rails 5 #3179

Merged
merged 5 commits into from
Sep 7, 2017
Merged

Update carrierwave to 1.0, which supports rails 5 #3179

merged 5 commits into from
Sep 7, 2017

Conversation

hackartisan
Copy link
Contributor

@projecthydra/sufia-code-reviewers

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) to 97.572% when pulling f2944ff on carrierwave into 78507ae on master.

@hackartisan hackartisan changed the title [WIP] Relax carrierwave dependency to allow for 1.0, which supports rails 5 Relax carrierwave dependency to allow for 1.0, which supports rails 5 Sep 7, 2017
@hackartisan hackartisan changed the title Relax carrierwave dependency to allow for 1.0, which supports rails 5 Update carrierwave to 1.0, which supports rails 5 Sep 7, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.596% when pulling 167a40a on carrierwave into 78507ae on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.596% when pulling a5d58f4 on carrierwave into 78507ae on master.

@mjgiarlo mjgiarlo merged commit 3489a14 into master Sep 7, 2017
@mjgiarlo mjgiarlo deleted the carrierwave branch September 7, 2017 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mjgiarlo mjgiarlo mjgiarlo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@hackartisan @coveralls @mjgiarlo @carolyncole