Skip to content

chore : Remove unwanted using statement #39176

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 38 commits into from
Jan 2, 2022
Merged

chore : Remove unwanted using statement #39176

merged 38 commits into from
Jan 2, 2022

Conversation

ShreyasJejurkar
Copy link
Contributor

@ShreyasJejurkar ShreyasJejurkar commented Dec 24, 2021

Just misc changes removed unwanted usings statement.
Area wise commits

Antiforgery - ff48cc4
Azure - 8e5527e
Caching - ced03ef
Components - 0f74df8
Configuration.KeyPerFile - 591988a
DataProtection - 663d1a3
DefaultBuilder - 0e5326c
Features - a9216cd
FileProviders - 26c8385
Framework - b8d524c
Hosting - 269a17a
Html.Abstractions - eb5487f
Http - d2a9480
Identity - 69ee974
JSInterop - 0d9683b
Localization - f0757c9
Logging.AzureAppServices - 2030a4c
Middleware - fecdbd2
Mvc - 9e05b18
ObjectPool - f14f952
Razor - eb1da37
Security - 952c66f
Servers - 415a773
Testing - 2df5857

@ghost ghost added the community-contribution Indicates that the PR has been added by a community member label Dec 24, 2021
@pranavkm
Copy link
Contributor

Could you turn enable this rule in the root editorconfig as part of this change: https://docs.microsoft.com/en-us/dotnet/fundamentals/code-analysis/style-rules/ide0005?

@ShreyasJejurkar
Copy link
Contributor Author

Yes, I am already going to do that, am just waiting for all green tests!

@ShreyasJejurkar
Copy link
Contributor Author

Hi @pranavkm I had to comment out the rule as of now because of some error that I was not able to resolve, so can you please fix some unwanted using with follow up commit and also uncomment rule in .editorconfig file!

@ShreyasJejurkar ShreyasJejurkar marked this pull request as ready for review December 25, 2021 13:34
@mkArtakMSFT mkArtakMSFT added area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates old-area-web-frameworks-do-not-use *DEPRECATED* This label is deprecated in favor of the area-mvc and area-minimal labels labels Dec 26, 2021
@mkArtakMSFT
Copy link
Contributor

Thank you for your PR, @ShreyasJejurkar.
@pranavkm I've assigned this to you for now. Feel free to reassign to others as needed. Thanks!

@pranavkm pranavkm enabled auto-merge (squash) January 2, 2022 14:39
@pranavkm pranavkm merged commit c85baf8 into dotnet:main Jan 2, 2022
@ghost ghost added this to the 7.0-preview1 milestone Jan 2, 2022
@danmoseley danmoseley removed the old-area-web-frameworks-do-not-use *DEPRECATED* This label is deprecated in favor of the area-mvc and area-minimal labels label Jul 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants