Skip to content

Update instructions and JwtBearer package versions#187

Merged
marcominerva merged 1 commit intomasterfrom
develop
Mar 11, 2026
Merged

Update instructions and JwtBearer package versions#187
marcominerva merged 1 commit intomasterfrom
develop

Conversation

@marcominerva
Copy link
Owner

Clarified guidance on ConfigureAwait usage in copilot-instructions.md and fixed duplicate test instruction. Upgraded Microsoft.AspNetCore.Authentication.JwtBearer to latest patch versions for .NET 8.0, 9.0, and 10.0 in csproj.

Clarified guidance on ConfigureAwait usage in copilot-instructions.md and fixed duplicate test instruction. Upgraded Microsoft.AspNetCore.Authentication.JwtBearer to latest patch versions for .NET 8.0, 9.0, and 10.0 in csproj.
Copilot AI review requested due to automatic review settings March 11, 2026 09:54
@marcominerva marcominerva merged commit 6ca4289 into master Mar 11, 2026
7 checks passed
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates internal contributor guidance and bumps JwtBearer dependency patch versions to keep the SimpleAuthentication abstractions aligned with current .NET servicing releases.

Changes:

  • Upgraded Microsoft.AspNetCore.Authentication.JwtBearer patch versions per target framework (net8.0, net9.0, net10.0).
  • Refined Copilot guidance on when ConfigureAwait(false) is appropriate (library vs. ASP.NET Core app context).
  • Fixed a formatting/duplication issue in the testing instructions list.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/SimpleAuthentication.Abstractions/SimpleAuthentication.Abstractions.csproj Updates JwtBearer package patch versions per target framework.
.github/copilot-instructions.md Clarifies ConfigureAwait(false) guidance and cleans up a duplicated/indented testing instruction line.

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.

2 participants