Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge master (dapper example) into openapi #1382

Closed
wants to merge 10 commits into from

Conversation

bkoelman
Copy link
Member

@bkoelman bkoelman commented Nov 8, 2023

TODO: Why the dramatic decrease in coverage? It looks like the test/OpenApiTests directory is now considered non-test code, possibly because test/OpenApiEndToEndTests.csproj has a dependency on it.

@bkoelman bkoelman changed the base branch from master to openapi November 8, 2023 00:05
Copy link

codecov bot commented Nov 8, 2023

Codecov Report

Merging #1382 (bcb0bbf) into openapi (b30030a) will increase coverage by 0.72%.
The diff coverage is 93.71%.

@@             Coverage Diff             @@
##           openapi    #1382      +/-   ##
===========================================
+ Coverage    90.27%   90.99%   +0.72%     
===========================================
  Files          308      382      +74     
  Lines        10159    12411    +2252     
  Branches      1693     1987     +294     
===========================================
+ Hits          9171    11294    +2123     
- Misses         651      723      +72     
- Partials       337      394      +57     
Files Coverage Δ
...pperExample/AtomicOperations/AmbientTransaction.cs 100.00% <100.00%> (ø)
.../DapperExample/Controllers/OperationsController.cs 100.00% <100.00%> (ø)
src/Examples/DapperExample/Data/RotatingList.cs 100.00% <100.00%> (ø)
src/Examples/DapperExample/Data/Seeder.cs 100.00% <100.00%> (ø)
...es/DapperExample/Definitions/TodoItemDefinition.cs 100.00% <100.00%> (ø)
...es/DapperExample/FromEntitiesNavigationResolver.cs 100.00% <100.00%> (ø)
...c/Examples/DapperExample/Models/AccountRecovery.cs 100.00% <100.00%> (ø)
src/Examples/DapperExample/Models/LoginAccount.cs 100.00% <100.00%> (ø)
src/Examples/DapperExample/Models/Person.cs 100.00% <100.00%> (ø)
src/Examples/DapperExample/Models/Tag.cs 100.00% <100.00%> (ø)
... and 70 more

... and 6 files with indirect coverage changes

@bkoelman
Copy link
Member Author

Superseded by #1388

@bkoelman bkoelman closed this Nov 12, 2023
@bkoelman bkoelman deleted the merge-master-into-openapi branch November 12, 2023 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant