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

fixes #67 #81

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

fixes #67 #81

wants to merge 2 commits into from

Conversation

romand
Copy link

@romand romand commented May 4, 2015

fixes #67
need it to resolve dependencies

@tillsc
Copy link

tillsc commented May 5, 2015

Wouldn't be >= 0.6.1 the better solution here?

@romand
Copy link
Author

romand commented May 5, 2015

why? it would allow 2.0.0, which may bring breaking changes (:

@tillsc
Copy link

tillsc commented May 5, 2015

>= 0.6.1, < 2.0.0 then. If dm-types works with 0.6.1 why forcing someone to update?

@romand
Copy link
Author

romand commented May 5, 2015

yep, I agree its better

@tillsc
Copy link

tillsc commented May 5, 2015

+1

@tpitale
Copy link
Member

tpitale commented Feb 14, 2016

@romand @tillsc If this is still valid (versions of safe_yaml, that is) I can merge it and try to get it in 1.3.0.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Upgrade to safe_yaml
3 participants
@romand @tillsc @tpitale