Skip to content

Conversation

@auduchinok
Copy link
Member

@auduchinok auduchinok commented Sep 1, 2022

Adds some APIs to FSharpEntity and FSharpType types that we've been using in our FCS fork.

@auduchinok auduchinok changed the title Symbols: add BasicQualifiedName, FSharpType.StrippedType Symbols: add more APIs Sep 1, 2022
@auduchinok auduchinok force-pushed the symbols-basicQualifiedName branch from ecaeaa5 to e0b96b7 Compare September 1, 2022 19:47
@auduchinok
Copy link
Member Author

This is ready.

dsyme
dsyme previously requested changes Sep 23, 2022
Copy link
Contributor

@dsyme dsyme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left some comments

@dsyme dsyme requested a review from a team as a code owner October 27, 2022 14:20
@dsyme
Copy link
Contributor

dsyme commented Oct 27, 2022

@T-Gro I tried to accept your changes but github is all confused

You should be able to push changes directly to the branch? Please do that, and let's get this green and in

@T-Gro T-Gro self-assigned this Nov 2, 2022
@T-Gro T-Gro added this to the November-2022 milestone Nov 2, 2022
@auduchinok
Copy link
Member Author

auduchinok commented Nov 3, 2022

Hey, sorry for not updating the PR earlier, I've had some local issues with FCS solution recently. I can update the needed things if CI is not green after this run

@T-Gro T-Gro requested review from T-Gro, dsyme and vzarytovskii November 3, 2022 19:10
@T-Gro
Copy link
Member

T-Gro commented Nov 3, 2022

Checks passing, please approve.

Copy link
Contributor

@psfinaki psfinaki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Eugene :)

@psfinaki
Copy link
Contributor

psfinaki commented Nov 4, 2022

@dsyme please reapprove

@T-Gro T-Gro dismissed dsyme’s stale review November 7, 2022 12:33

Incorporated

@T-Gro T-Gro merged commit 8ba5a40 into dotnet:main Nov 7, 2022
@dsyme
Copy link
Contributor

dsyme commented Nov 7, 2022

Thanks @psfinaki @T-Gro !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants