Skip to content

[WIP] Mesos 0.23 #5

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.

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

Merged
merged 28 commits into from
Oct 29, 2015
Merged

[WIP] Mesos 0.23 #5

merged 28 commits into from
Oct 29, 2015

Conversation

jhedev
Copy link
Collaborator

@jhedev jhedev commented Aug 31, 2015

I've tried to make this project work with mesos 0.23.

I guess, I'm pretty much a noob with regard to Haskell/C++ interop, but I gave my best and all tests are passing already \o/. However, I'm open for any criticism/improvements/suggestions :)

The TestFramework also works with a local running Mesos 0.23 instance.

Still to do: Cleanup the code (especially formatting and comments)

@iand675
Copy link
Owner

iand675 commented Oct 14, 2015

@jhedev Hey, this is great! I totally missed the initial submission. Would you still consider it a WIP? Otherwise I'm pretty good with straight-up merging it.

@iand675
Copy link
Owner

iand675 commented Oct 14, 2015

p.s., I'm adding you as a contributor, so if you want to push changes in the future, feel free to do them directly in this repo.

@jhedev
Copy link
Collaborator Author

jhedev commented Oct 14, 2015

Well, I actually planned to clean up the code a bit, especially the formatting, which is quite messy at some points. However, I've been very busy with university work lately and I guess it will take at least a few weeks until I have time for that.

Other than that there is nothing to be added or changed. So if that's not an issue, go ahead and merge it :) And I will improve the code/documentation as soon as I have some time.

iand675 added a commit that referenced this pull request Oct 29, 2015
@iand675 iand675 merged commit 0ff54fc into iand675:master Oct 29, 2015
@jhedev jhedev deleted the mesos-0.23 branch October 30, 2015 06:44
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.

2 participants