Skip to content

Add accepted_resource_role kwarg for Marathon 0.9.0 support#48

Merged
solarkennedy merged 3 commits intothefactory:masterfrom
keshavdv:master
Jul 21, 2015
Merged

Add accepted_resource_role kwarg for Marathon 0.9.0 support#48
solarkennedy merged 3 commits intothefactory:masterfrom
keshavdv:master

Conversation

@keshavdv
Copy link
Copy Markdown

No description provided.

@solarkennedy
Copy link
Copy Markdown
Contributor

Whoa!

@solarkennedy
Copy link
Copy Markdown
Contributor

Well, the tests pass! If there is any other api-breaking change on 0.9.0, then we are not itesting against them so we won't know yet! This was exactly the thing that made me comment out 0.9.0 before, it is nice to see this was the only thing (so far) that was preventing this.

Can you update the README.md + CHANGELOG as well and bump the version number (but note it as not-released yet)

@mrtheb
Copy link
Copy Markdown
Contributor

mrtheb commented Jul 20, 2015

thanks, I needed this too

solarkennedy added a commit that referenced this pull request Jul 21, 2015
Add accepted_resource_role kwarg for Marathon 0.9.0 support
@solarkennedy solarkennedy merged commit a3fd966 into thefactory:master Jul 21, 2015
@solarkennedy
Copy link
Copy Markdown
Contributor

Thanks @keshavdv !

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.

3 participants