Skip to content

Globally disable implicit namespaces#56348

Merged
safern merged 1 commit into
dotnet:mainfrom
safern:DisableImplNamespaces
Jul 27, 2021
Merged

Globally disable implicit namespaces#56348
safern merged 1 commit into
dotnet:mainfrom
safern:DisableImplNamespaces

Conversation

@safern

@safern safern commented Jul 27, 2021

Copy link
Copy Markdown
Member

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

ghost commented Jul 27, 2021

Copy link
Copy Markdown

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

ghost commented Jul 27, 2021

Copy link
Copy Markdown

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
Copy Markdown
Member

thanks

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

@ViktorHofer ViktorHofer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@dotMorten

dotMorten commented Jul 28, 2021

Copy link
Copy Markdown

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

@danmoseley

Copy link
Copy Markdown
Member

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

@safern

safern commented Jul 28, 2021

Copy link
Copy Markdown
Member Author

@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