Remove unused variable

[mthalman]

When using the latest build of .NET 8 Preview 4 SDK to build the aspnetcore repo, it results in the following analyzer error:

/repos/dotnet/src/aspnetcore/artifacts/source-build/self/src/src/Tools/dotnet-user-jwts/src/Commands/ListCommand.cs(55,45): error IDE0059: Unnecessary assignment of a value to 'jwts' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0059) [/repos/dotnet/src/aspnetcore/artifacts/source-build/self/src/src/Tools/dotnet-user-jwts/src/dotnet-user-jwts.csproj]

This was caught by the Source-build bootstrapping build workflow which uses a source-built version of the latest SDK to build the product's repos.

Fixed by removing the unused variable.

[davidfowl]

cc @captainsafia we should have these warnings turned on in our build too.

[mitchdenny]

I think this actually broke the build. The variable was used - I think this might have just been a sequencing issue with the SDK update PR when it came in. If you look at the source on main the jwts variable is used:

aspnetcore/src/Tools/dotnet-user-jwts/src/Commands/ListCommand.cs

Line 57 in ab1bfa7

reporter.Output(JsonSerializer.Serialize(jwts, new JsonSerializerOptions { WriteIndented = true }));

That code came into being in the Update SDK PR that was merged today.

[ghost]

Hi @mitchdenny. It looks like you just commented on a closed PR. The team will most probably miss it. If you'd like to bring something important up to their attention, consider filing a new issue and add enough details to build context.

[davidfowl]

oops the build was green no?

[davidfowl]

#47944