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 a C# wrapper for convenience #75

Closed
wants to merge 1 commit into from

Conversation

baronfel
Copy link
Contributor

This adds a new project for authoring a C# wrapper that handles nice interop around IEnumerable<Test> instead of Test list and Task instead of Async<unit> for our C# friends. There is also a small sample test project to verify that things work.

Is this valued? If so, what needs to be done at this point to author packages?

@haf
Copy link
Owner

haf commented Jan 11, 2017

I'm going to make the executive decision here and say that I want expecto to be F# only in its core repository, but I wouldn't mind if you publish this as a nuget.

If you use albacore like is done here you don't need to do anything to publish the new package. It's all automatic and you just run 'bundle exec rake release'.

@forki
Copy link
Contributor

forki commented Jan 11, 2017

Can't you write the same thing in F# and put it in the core repo? Then we would only have the test csproj - which I believe would be a good thing. I personally don't think writing tests in C# is useful in any way, but it's probably still a good thing to attract more people.

@baronfel
Copy link
Contributor Author

I actually tried to write the csharp interop layer in F# first, but ran into a few issues with exposing the static member functions. Though now I see the Extension attribute, so I'll give that a try in the morning.

@haf
Copy link
Owner

haf commented Jan 11, 2017

@haf
Copy link
Owner

haf commented Jan 12, 2017

Perhaps you can take a leaf from the above link?

@baronfel
Copy link
Contributor Author

Yes that looks to be exactly what I need. I'll try to get to it today.

@haf
Copy link
Owner

haf commented Jan 25, 2017

I'm closing this in the mean while. Tell me if you need any help.

@haf haf closed this Jan 25, 2017
@baronfel baronfel mentioned this pull request Feb 23, 2017
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