Skip to content

Conversation

@kerams
Copy link
Contributor

@kerams kerams commented Jul 29, 2023

Removing completions that are not usable in a pattern when dotting into a type, module or namespace - methods, properties, values that are not literals and fields that are not constants.

Before

devenv_Ytw2ETIA72

After

devenv_dPKVHMzYvj

(The 2nd screenshot contains extra members because of TFM difference)

@kerams kerams requested a review from a team as a code owner July 29, 2023 21:48
@kerams
Copy link
Contributor Author

kerams commented Jul 30, 2023

Also, what do you think about about classifying IL constants the same way as F# literals? The screenshot below is from a popular .NET language.

image

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, this is useful!

@psfinaki
Copy link
Contributor

Also, what do you think about classifying IL constants the same way as F# literals?

I am on the fence here, maybe you can submit a separate PR to bring this into a discussion?

@vzarytovskii
Copy link
Member

Also, what do you think about about classifying IL constants the same way as F# literals? The screenshot below is from a popular .NET language.

image

I don't see why not, after all x-project solutions are not unpopular, amirite?

@0101 0101 merged commit 950f811 into dotnet:main Jul 31, 2023
@kerams kerams deleted the c branch July 31, 2023 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants