Skip to content

Remove global:: from generated global usings #44916

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
merged 2 commits into from
Nov 22, 2024

Conversation

jjonescz
Copy link
Member

Fixes #42479.

@dotnet-issue-labeler dotnet-issue-labeler bot added the untriaged Request triage from a team member label Nov 18, 2024
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

1 similar comment
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@jjonescz jjonescz marked this pull request as ready for review November 18, 2024 13:17
@jjonescz jjonescz requested a review from a team November 18, 2024 13:18
@@ -103,6 +103,7 @@ public void It_can_generate_custom_usings()
<Using Include=""CustomNamespace"" />
<Using Include=""TestStaticNamespace"" Static=""True"" />
<Using Include=""System.Biology"" Alias=""AppliedChemistry"" />
<Using Include=""(System.String X, int Y)"" Alias=""MyTuple"" />
Copy link
Member

Choose a reason for hiding this comment

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

Let's add a direct keyword example, like making an alias for int or dynamic.

@jjonescz
Copy link
Member Author

@jaredpar @marcpopMSFT I'm not sure what reviews do I need for this one

@jaredpar
Copy link
Member

I'm not sure what reviews do I need for this one

It just needs sign off from us so when you're ready merge.

@jjonescz jjonescz merged commit d7fe6e6 into dotnet:main Nov 22, 2024
37 checks passed
@jjonescz jjonescz deleted the 42479-global-usings branch November 22, 2024 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-ImplicitUsings untriaged Request triage from a team member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support ValueTuple in <Using /> element
3 participants