Skip to content
This repository was archived by the owner on Apr 17, 2018. It is now read-only.

Loosen dependency on multi_json #85

Closed
wants to merge 3 commits into from
Closed

Conversation

basex
Copy link

@basex basex commented May 25, 2016

The other datamapper gems depend mostly on multi_json 1.3.2
Example: https://github.com/datamapper/dm-serializer/blob/master/dm-serializer.gemspec

dm-types also depended previously on 1.3.2 e541869

@tpitale
Copy link
Member

tpitale commented May 25, 2016

After I merge this #86 can you rebase/squash and push this PR again so that it runs in CI (now that I enabled PR runs again)?

@basex
Copy link
Author

basex commented May 25, 2016

yes I can do that, I also relaxed the multi_json dependency on dm-serializer
datamapper/dm-serializer#46

@tpitale
Copy link
Member

tpitale commented May 25, 2016

Go for a rebase, I just merged the fix into master. Thanks @basex!

@basex
Copy link
Author

basex commented May 25, 2016

Sorry tpitale, there is not really a need to decrease this gem dm-types version required. The problem is with dm-serializer. I will rebase or create a new PR there.

@basex basex closed this May 25, 2016
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.

2 participants