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

F# samples #824

Merged
merged 2 commits into from
Sep 1, 2023
Merged

F# samples #824

merged 2 commits into from
Sep 1, 2023

Conversation

64J0
Copy link
Contributor

@64J0 64J0 commented Jul 4, 2023

Description

This PR is intended to present an example for the translation of the samples from C# to F#.

My goal is to encourage F# people to use this project, especially the newcomers with no previous experience in .NET, by showing how it can be used with new sample programs (basically translating the current C# samples to F#).

It's related to this issue: #822.

How to test

Make sure that you have .NET SDK version 7 installed.

cd YamlDotNet.Samples.Fsharp/
dotnet run
# check the output

@64J0
Copy link
Contributor Author

64J0 commented Jul 4, 2023

Running in my environment:

image

@EdwardCooke
Copy link
Collaborator

Is there any reason in particular that you didn't add this samples project to the solution?

@64J0
Copy link
Contributor Author

64J0 commented Aug 14, 2023

Yes @EdwardCooke, I'm not sure how it is going to impact the project. Do you think it's better to add it there?

@EdwardCooke
Copy link
Collaborator

It would be best to have it in the solution file so it's visible when opening it up in Visual Studio

@64J0
Copy link
Contributor Author

64J0 commented Aug 29, 2023

Got it, I'll add it there soon.

@64J0
Copy link
Contributor Author

64J0 commented Aug 29, 2023

Added right now.

@EdwardCooke EdwardCooke merged commit e856ff3 into aaubry:master Sep 1, 2023
@64J0 64J0 deleted the 64J0-patch-1 branch September 1, 2023 16:45
@aaubry
Copy link
Owner

aaubry commented Sep 20, 2023

This feature has been released in version 13.4.0.

@64J0
Copy link
Contributor Author

64J0 commented Sep 20, 2023

Nice!

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