-
Notifications
You must be signed in to change notification settings - Fork 831
C# collection expression support for F# lists & sets #17359
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
vzarytovskii
merged 33 commits into
dotnet:main
from
brianrourkeboll:csharp-collection-expression-support
Jul 16, 2024
Merged
Changes from 2 commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
c76c603
C# collection expression support for lists & sets
brianrourkeboll b2ef7a9
Add trailing newline
brianrourkeboll b4facf9
These should be `internal`
brianrourkeboll 69410bc
Cleanup
brianrourkeboll 1700130
Update FSharp.Core surface area
brianrourkeboll 5699cb9
Update release notes
brianrourkeboll 21b61a1
Meant to use that
brianrourkeboll 29ab91e
Remove redundant attribute
brianrourkeboll d84ab1c
Add doc comments
brianrourkeboll 4c3ab92
Missed that test name
brianrourkeboll 7d114f8
Add runner JSON
brianrourkeboll 94a8541
Replace ifdef with comment
brianrourkeboll d93e696
Add direct tests for non-generic List & Set types
brianrourkeboll a0de93d
Merge branch 'main' of https://github.com/dotnet/fsharp into csharp-c…
brianrourkeboll eb0e1a2
Link RFC PR
brianrourkeboll fca5827
Make test project work
brianrourkeboll af5779c
Merge branch 'main' of https://github.com/dotnet/fsharp into csharp-c…
brianrourkeboll 6368265
Copy/paste bug
brianrourkeboll 44e0dcd
Probably also not
brianrourkeboll e4a1f08
Not that either
brianrourkeboll b0e026e
Merge branch 'main' of https://github.com/dotnet/fsharp into csharp-c…
brianrourkeboll 57d6ec6
Merge branch 'main' of https://github.com/dotnet/fsharp into csharp-c…
brianrourkeboll 976f140
Add C# 12 language version case
brianrourkeboll 2d0f88f
Add interop tests
brianrourkeboll e5e67c8
Remove separate C# test project
brianrourkeboll a55e7aa
Merge branch 'main' of https://github.com/dotnet/fsharp into csharp-c…
brianrourkeboll edc1db7
Remove refs to removed test project
brianrourkeboll 8892229
Move release notes
brianrourkeboll 9fcaf8a
Merge branch 'main' of https://github.com/dotnet/fsharp into csharp-c…
brianrourkeboll bf1a1b7
Update trimmed size
brianrourkeboll 24d1df6
Need span
brianrourkeboll d358314
Update trimmed size
brianrourkeboll 860ab08
Merge branch 'main' into csharp-collection-expression-support
brianrourkeboll File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
23 changes: 23 additions & 0 deletions
23
tests/FSharp.Core.UnitTests.CSharp/FSharp.Core.UnitTests.CSharp.csproj
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,23 @@ | ||
| <Project Sdk="Microsoft.NET.Sdk"> | ||
|
|
||
| <PropertyGroup> | ||
| <TargetFrameworks>$(FSharpNetCoreProductTargetFramework);net472</TargetFrameworks> | ||
| <TargetFrameworks Condition="'$(OS)' == 'Unix'">$(FSharpNetCoreProductTargetFramework)</TargetFrameworks> | ||
| <ImplicitUsings>enable</ImplicitUsings> | ||
| <Nullable>enable</Nullable> | ||
| <LangVersion>12.0</LangVersion> | ||
| <UnitTestType>xunit</UnitTestType> | ||
| <IsPackable>false</IsPackable> | ||
| <IsTestProject>true</IsTestProject> | ||
| </PropertyGroup> | ||
|
|
||
| <ItemGroup> | ||
| <ProjectReference Include="$(FSharpSourcesRoot)\FSharp.Core\FSharp.Core.fsproj" /> | ||
| <ProjectReference Include="$(FSharpSourcesRoot)\..\tests\FSharp.Test.Utilities\FSharp.Test.Utilities.fsproj" /> | ||
| </ItemGroup> | ||
|
|
||
| <ItemGroup> | ||
| <Using Include="Xunit" /> | ||
| </ItemGroup> | ||
|
|
||
| </Project> |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.