Closed DavidAllison closed 9 years ago
@dkubb Not sure if you saw the note above - I think this PR's single line change couldn't have an effect on whether dm-migrations
can be found, since all it does is upgrade safe_yaml
. Any thoughts?
@dkubb Any chance you might be able to take a look at the one line PR above?
@DavidAllison It appears as though the safe_yaml conflicts with other dependencies: https://travis-ci.org/datamapper/dm-types/jobs/13220738 in the case of dm-yaml-adapter. Any thoughts on resolving that?
This is the simplest way I could see to make this change (though I've noted that in
dm_types
master branchsafe_yaml
is actually gone, so if it's preferable to just remove the requirement let me know).I found that running bundle on the branch fails due to
dm-migrations
not being found, but this issue pre-exists this change.It looks like this, for reference: