Skip to content

Conversation

MangelMaxime
Copy link
Member

I am starting this PR early to avoid duplicate work regarding nullable/nullness support #3887

@MangelMaxime MangelMaxime marked this pull request as draft January 22, 2025 13:00
@ncave
Copy link
Collaborator

ncave commented Jan 22, 2025

@MangelMaxime I would suggest doing separate PRs, for supporting nullable value type (i.e. System.Nullable<T>) and a different PR for nullable ref types (T | null), and keeping them as small as possible to be able to iterate on them.

Perhaps this PR can be renamed to Added support for FSharp.Linq.Nullable conversions.

@MangelMaxime
Copy link
Member Author

I would suggest doing separate PRs,

I agree this PR will need to be split because locally I have even more different fixes for others part of Fable because I am discovering some unsupported APIs.

I will make specialized PRs for the different changes.

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