Skip to content

Conversation

@sungam3r
Copy link
Member

@sungam3r sungam3r requested a review from Shane32 December 25, 2021 17:42
@github-actions github-actions bot added the test label Dec 25, 2021
@sungam3r sungam3r added BREAKING Breaking changes in either public API or runtime behavior bugfix Pull request that fixes a bug labels Dec 25, 2021
@sungam3r sungam3r added this to the v6 milestone Dec 25, 2021
@sungam3r sungam3r removed the test label Dec 25, 2021
@github-actions github-actions bot added CI CI configuration issue or pull request test labels Dec 25, 2021
@sungam3r sungam3r removed the test label Dec 25, 2021
@sungam3r sungam3r self-assigned this Dec 25, 2021
…port

# Conflicts:
#	.github/workflows/test.yml
#	samples/Samples.Server/Startup.cs
#	samples/Samples.Server/StartupWithRouting.cs
#	src/All/All.csproj
#	src/Authorization.AspNetCore/AuthorizationValidationRule.cs
#	src/Core/Core.csproj
#	src/Transports.AspNetCore.NewtonsoftJson/Transports.AspNetCore.NewtonsoftJson.csproj
#	src/Transports.AspNetCore.SystemTextJson/GraphQLRequestDeserializer.cs
#	src/Transports.AspNetCore.SystemTextJson/Transports.AspNetCore.SystemTextJson.csproj
#	src/Transports.Subscriptions.Abstractions/Transports.Subscriptions.Abstractions.csproj
@github-actions github-actions bot added the test label Dec 28, 2021
@sungam3r
Copy link
Member Author

Ready for review.

@sungam3r sungam3r merged commit 09e9278 into develop Dec 29, 2021
@sungam3r sungam3r deleted the auth-port branch December 29, 2021 13:49
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 26.66667% with 55 lines in your changes missing coverage. Please review.
✅ Project coverage is 49.17%. Comparing base (46e5fa4) to head (b5ffb48).
⚠️ Report is 234 commits behind head on develop.

Files with missing lines Patch % Lines
...rization.AspNetCore/AuthorizationValidationRule.cs 26.08% 46 Missing and 5 partials ⚠️
.../Extensions/GraphQLBuilderUserContextExtensions.cs 0.00% 3 Missing ⚠️
src/Core/DefaultGraphQLExecuter.cs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #676      +/-   ##
===========================================
- Coverage    50.85%   49.17%   -1.68%     
===========================================
  Files           67       67              
  Lines         1815     1881      +66     
  Branches       183      196      +13     
===========================================
+ Hits           923      925       +2     
- Misses         838      898      +60     
- Partials        54       58       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BREAKING Breaking changes in either public API or runtime behavior bugfix Pull request that fixes a bug CI CI configuration issue or pull request test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants