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 initial support for Fuchu tests #1268

Merged
merged 1 commit into from
Jun 19, 2016
Merged

Add initial support for Fuchu tests #1268

merged 1 commit into from
Jun 19, 2016

Conversation

Kazark
Copy link
Contributor

@Kazark Kazark commented Jun 15, 2016

Something like this has been discussed at #234, and there is an open issue for this more specifically at #742. And while there seem to be some questions of how to do this in the best way, or to polish Fuchu itself, I am of the opinion that it is better to have something than nothing. It is rather reassuring to n00bs particularly when they see that tools are at least aware of each other, and this is one of the big draws of FAKE.

The code is derived from the example from Fuchu itself. Note that I have not tested this on Mono myself; I was trusting it to work correctly since it came from Fuchu. I have however tested it on Windows. If you want me to test it on Linux, I'm happy to do so.

Something like this has been discussed at fsprojects#234, and there is an open
issue for this more specifically at fsprojects#742. And while there seem to be
some questions of how to do this in the best way, or polish it, I am of
the opinion that it is better to have something than nothing. It is
rather reassuring to n00bs particularly when they see that tools are at
least aware of each other, and this is one of the big draws of FAKE.

The code is derived from the example from Fuchu itself:

    https://github.com/mausch/Fuchu/blob/5ef9ec303abf56f8ca5511cda3fb53bfa705007c/build.fsx#L119
@@ -0,0 +1,19 @@
module Fake.FuchuHelper
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should I have marked this [<AutoOpen>]?

@forki forki merged commit fd7d1fc into fsprojects:master Jun 19, 2016
@forki
Copy link
Member

forki commented Jun 19, 2016

cool thanks

@Kazark Kazark deleted the fuchu branch June 20, 2016 12:07
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