Skip to content

Conversation

@jaredpar
Copy link
Member

@jaredpar jaredpar commented Apr 8, 2025

This change removes SCI from the source-generators directory. This dependency is always resolved by the C# compiler that is loading the generator. Deploying it is just eating up disk space.

This change removes SCI from the source-generators directory. This
dependency is always resolved by the C# compiler that is loading the
generator. Deploying it is just eating up disk space.
@jaredpar jaredpar requested a review from a team as a code owner April 8, 2025 16:26
<ItemGroup>
<Content Include="$(ArtifactsBinDir)Microsoft.CodeAnalysis.Razor.Compiler\$(Configuration)\netstandard2.0\*.*" Exclude="$(ArtifactsBinDir)**\*.pdb" PackagePath="\source-generators" />
<Content Include="$(ArtifactsBinDir)Microsoft.CodeAnalysis.Razor.Tooling.Internal\$(Configuration)\netstandard2.0\Microsoft.Extensions.ObjectPool.dll" PackagePath="\source-generators" />
<Content Include="$(ArtifactsBinDir)Microsoft.CodeAnalysis.Razor.Tooling.Internal\$(Configuration)\netstandard2.0\System.Collections.Immutable.dll" PackagePath="\source-generators" />
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, good catch

@jaredpar jaredpar merged commit c9ee435 into main Apr 8, 2025
11 checks passed
@jaredpar jaredpar deleted the dev/jaredpar/sci branch April 8, 2025 17:33
@dotnet-policy-service dotnet-policy-service bot added this to the Next milestone Apr 8, 2025
@ryzngard ryzngard mentioned this pull request Apr 18, 2025
@ryzngard ryzngard mentioned this pull request Apr 28, 2025
@RikkiGibson RikkiGibson modified the milestones: Next, 18.0 P1 Aug 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants