Skip to content

Conversation

@nojaf
Copy link
Contributor

@nojaf nojaf commented Feb 2, 2023

I'd like to get rid of all the ? parameters and work with overloads instead.
I deliberately skipped FormatDocumentAsync as Alex mentioned he would pick this up.

@nojaf nojaf requested a review from a team February 2, 2023 08:30
@nojaf nojaf merged commit 7ca19ab into fsprojects:v6.0 Feb 3, 2023
nojaf added a commit that referenced this pull request Feb 3, 2023
* Remove F# option from public API.

* Apply suggestions from code review

Co-authored-by: dawe <[email protected]>

---------

Co-authored-by: dawe <[email protected]>
nojaf added a commit that referenced this pull request Feb 4, 2023
* Remove F# option from public API.

* Apply suggestions from code review

Co-authored-by: dawe <[email protected]>

---------

Co-authored-by: dawe <[email protected]>
nojaf added a commit that referenced this pull request Feb 22, 2023
* Remove F# option from public API.

* Apply suggestions from code review

Co-authored-by: dawe <[email protected]>

---------

Co-authored-by: dawe <[email protected]>
nojaf added a commit that referenced this pull request Mar 17, 2023
* Remove F# option from public API.

* Apply suggestions from code review

Co-authored-by: dawe <[email protected]>

---------

Co-authored-by: dawe <[email protected]>
nojaf added a commit that referenced this pull request Mar 27, 2023
* Remove F# option from public API.

* Apply suggestions from code review

Co-authored-by: dawe <[email protected]>

---------

Co-authored-by: dawe <[email protected]>
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