Skip to content
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

Add RoundhouseHelper #456

Merged
merged 1 commit into from Jun 6, 2014
Merged

Add RoundhouseHelper #456

merged 1 commit into from Jun 6, 2014

Conversation

ChrisMissal
Copy link
Contributor

I'd like to get this added, I've been using an almost identical version locally and it works great. Let me know if there's anything more you need from me on this pull request.


traceStartTask "Roundhouse" args

if -1 = ExecProcess (fun info ->
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should probably check for <>0

@forki
Copy link
Member

forki commented Jun 4, 2014

Very cool.

@ChrisMissal
Copy link
Contributor Author

Good call! Updated.

forki added a commit that referenced this pull request Jun 6, 2014
@forki forki merged commit 9d55328 into fsprojects:master Jun 6, 2014
@forki
Copy link
Member

forki commented Jun 6, 2014

I will release this after the weekend when I'm home again.

Thanks again. Good work

@ChrisMissal ChrisMissal deleted the roundhouse branch June 6, 2014 14:32
@ChrisMissal
Copy link
Contributor Author

Great, thanks!

@ctaggart
Copy link
Contributor

ctaggart commented Jun 8, 2014

Yes, @ChrisMissal thanks! Nice to see you using FAKE. At some point I may have to present FAKE to the newly formed Austin F# Users Group.

@ChrisMissal
Copy link
Contributor Author

For sure, I just joined the group!

@panesofglass
Copy link
Contributor

@ChrisMissal does this mean we really can migrate all the WebApiContrib projects to FAKE now? ;)

@ChrisMissal
Copy link
Contributor Author

Well now that I have some experience using it, sure, if you're offering 😄

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.

None yet

4 participants