Update NuGet packages and remove copilot instructions file#186
Update NuGet packages and remove copilot instructions file#186marcominerva merged 1 commit intomasterfrom
Conversation
Updated Microsoft.AspNetCore.OpenApi, Swashbuckle.AspNetCore, and SimpleAuthenticationTools.Abstractions to latest patch versions across sample and main projects. Also removed .github/copilot-instructions.md from solution items.
There was a problem hiding this comment.
Pull request overview
Updates NuGet package patch versions across the main library projects and multiple sample projects, and removes the Copilot instructions markdown file from the solution’s “Solution Items” list.
Changes:
- Bump
SimpleAuthenticationTools.Abstractionsto3.1.9in main projects. - Bump Swashbuckle packages to
10.1.2andMicrosoft.AspNetCore.OpenApito10.0.3in samples. - Remove
.github/copilot-instructions.mdfromSimpleAuthentication.slnxsolution items.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/SimpleAuthentication/SimpleAuthentication.csproj | Updates SimpleAuthenticationTools.Abstractions package version. |
| src/SimpleAuthentication.Swashbuckle/SimpleAuthentication.Swashbuckle.csproj | Updates SimpleAuthenticationTools.Abstractions and Swashbuckle.AspNetCore.SwaggerGen package versions. |
| samples/MinimalApis/Net8JwtBearerSample/Net8JwtBearerSample.csproj | Updates Swashbuckle.AspNetCore package version. |
| samples/MinimalApis/JwtBearerSample/JwtBearerSample.csproj | Updates Microsoft.AspNetCore.OpenApi and Swashbuckle.AspNetCore.SwaggerUI package versions. |
| samples/MinimalApis/BasicAuthenticationSample/BasicAuthenticationSample.csproj | Updates Microsoft.AspNetCore.OpenApi and Swashbuckle.AspNetCore.SwaggerUI package versions. |
| samples/MinimalApis/ApiKeySample/ApiKeySample.csproj | Updates Microsoft.AspNetCore.OpenApi and Swashbuckle.AspNetCore.SwaggerUI package versions. |
| samples/Controllers/JwtBearerSample/JwtBearerSample.csproj | Updates Microsoft.AspNetCore.OpenApi and Swashbuckle.AspNetCore.SwaggerUI package versions. |
| samples/Controllers/BasicAuthenticationSample/BasicAuthenticationSample.csproj | Updates Microsoft.AspNetCore.OpenApi and Swashbuckle.AspNetCore.SwaggerUI package versions. |
| samples/Controllers/ApiKeySample/ApiKeySample.csproj | Updates Microsoft.AspNetCore.OpenApi and Swashbuckle.AspNetCore.SwaggerUI package versions. |
| SimpleAuthentication.slnx | Removes .github/copilot-instructions.md from solution items. |
| <PackageReference Include="Microsoft.AspNetCore.OpenApi" Version="10.0.2" /> | ||
| </ItemGroup> | ||
|
|
||
| <ItemGroup> | ||
| <PackageReference Include="SimpleAuthenticationTools.Abstractions" Version="3.1.7" /> | ||
| <PackageReference Include="SimpleAuthenticationTools.Abstractions" Version="3.1.9" /> |
There was a problem hiding this comment.
In this project, the net10.0 conditional dependency still pins Microsoft.AspNetCore.OpenApi to 10.0.2, while the samples were updated to 10.0.3. If the intent is to update OpenApi across main + samples (as the PR description suggests), bump this net10.0 reference as well (or explain why it must stay on 10.0.2).
| <Folder Name="/Solution Items/"> | ||
| <File Path=".editorconfig" /> | ||
| <File Path=".github/copilot-instructions.md" /> | ||
| <File Path="src/Directory.Build.props" /> |
There was a problem hiding this comment.
This change only removes .github/copilot-instructions.md from the solution items, but the file still exists in the repo. The PR title implies the instructions file was removed; consider either deleting the file as well, or adjusting the title/description to match the actual change.
Updated Microsoft.AspNetCore.OpenApi, Swashbuckle.AspNetCore, and SimpleAuthenticationTools.Abstractions to latest patch versions across sample and main projects. Also removed .github/copilot-instructions.md from solution items.