Provide analyzer for removing unneeded public partial class Program by captainsafia · Pull Request #58482 · dotnet/aspnetcore (original) (raw)

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 andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation23 Commits5 Checks27 Files changed

Conversation

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

captainsafia

Follow-up to #58199.

Based on the code generation that we introduced earlier, users can get rid of the explicit public partial class Program { } declarations in their source code and rely on the new default behavior.

This PR introduces an analyzer to find these explicit declarations and a code fixer to remove them.

Implements #58488

@ghost ghost added the area-infrastructure

Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework

label

Oct 17, 2024

@captainsafia captainsafia added analyzer

Indicates an issue which is related to analyzer experience

feature-mvc-testing

MVC testing package

area-minimal

Includes minimal APIs, endpoint filters, parameter binding, request delegate generator etc

and removed area-infrastructure

Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework

labels

Oct 17, 2024

@captainsafia

martincostello

@captainsafia

martincostello

Choose a reason for hiding this comment

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

:shipit:

david-acker

gfoidl

amcasey

@captainsafia

BrennanConroy

@captainsafia

mgravell

mgravell

BrennanConroy

@captainsafia

mgravell

captainsafia added a commit that referenced this pull request

Feb 11, 2025

@captainsafia

…58482)

Labels

analyzer

Indicates an issue which is related to analyzer experience

area-minimal

Includes minimal APIs, endpoint filters, parameter binding, request delegate generator etc

feature-mvc-testing

MVC testing package