Skip to content

Conversation

@safern
Copy link
Member

@safern safern commented Jul 27, 2021

We've hit more instances of this, now when building MSBuild tasks as the MSBuild Task type conflicts with System.Threading.Tasks.Task type which its namespace is implicitly defined.

@safern safern requested a review from a team July 27, 2021 00:35
@ghost
Copy link

ghost commented Jul 27, 2021

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.

@ghost
Copy link

ghost commented Jul 27, 2021

Hello @safern!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@danmoseley
Copy link
Member

thanks

@safern safern merged commit 7545f98 into dotnet:main Jul 27, 2021
@safern safern deleted the DisableImplNamespaces branch July 27, 2021 06:22
Copy link
Member

@ViktorHofer ViktorHofer left a comment

Choose a reason for hiding this comment

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

LGTM

@dotMorten
Copy link

dotMorten commented Jul 28, 2021

Been seeing several PRs like this today. Maybe.... just maybe auto generating global namespaces isn't such a great feature?

@danmoseley
Copy link
Member

@MadsTorgersen where is the right place for folks to give feedback on the implicit usings feature?

@safern
Copy link
Member Author

safern commented Jul 28, 2021

@danmoseley I believe it is in the SDK. It was added here: dotnet/sdk#18459

@ghost ghost locked as resolved and limited conversation to collaborators Aug 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants