Skip to content

Merge master v5.3.0 into openapi #1293

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

Merged
merged 48 commits into from
Jul 25, 2023
Merged

Conversation

bkoelman
Copy link
Member

@bkoelman bkoelman commented Jul 23, 2023

This PR merges the latest master branch (v5.3.0) into the openapi branch.

Commits to review (the rest originates from merge): the last 4 commits.

QUALITY CHECKLIST

  • Changes implemented in code
  • Complies with our contributing guidelines
  • N/A: Adapted tests
  • N/A: Documentation updated

bkoelman and others added 30 commits May 14, 2023 23:05
The new docfx template no longer produces a /styles directory, therefore it needs to be created first for the PowerShell homepage copy to succeed.

https://stackoverflow.com/questions/2695504/powershell-2-copy-item-which-creates-a-folder-if-doesnt-exist

Also, the allyoucan.cloud link is unreachable, which blocks loading the home page for 20 seconds. Removed the reference, not sure if it was needed.
Corrected HTTP method in updating.md
Fix exception thrown by EF Core 8 preview
…shape of parsers, making it easier to use them directly from external code
@codecov
Copy link

codecov bot commented Jul 24, 2023

Codecov Report

Merging #1293 (94c7a33) into openapi (adff2a9) will increase coverage by 4.20%.
The diff coverage is 93.33%.

❗ Current head 94c7a33 differs from pull request most recent head 313864b. Consider uploading reports for the commit 313864b to get more accurate results

@@             Coverage Diff             @@
##           openapi    #1293      +/-   ##
===========================================
+ Coverage    92.61%   96.81%   +4.20%     
===========================================
  Files          316       21     -295     
  Lines         9313      597    -8716     
===========================================
- Hits          8625      578    -8047     
+ Misses         688       19     -669     
Impacted Files Coverage Δ
...tityFrameworkExample/Repositories/TagRepository.cs 0.00% <0.00%> (ø)
...ntityFrameworkExample/QueryLayerToLinqConverter.cs 100.00% <100.00%> (ø)
...Example/Repositories/InMemoryResourceRepository.cs 100.00% <100.00%> (ø)
...yFrameworkExample/Repositories/PersonRepository.cs 100.00% <100.00%> (ø)
...rameworkExample/Repositories/TodoItemRepository.cs 100.00% <100.00%> (ø)
...ameworkExample/Services/InMemoryResourceService.cs 93.42% <100.00%> (ø)
...EntityFrameworkExample/Services/TodoItemService.cs 77.77% <100.00%> (ø)
...ApiDotNetCore.SourceGenerators/SourceCodeWriter.cs 97.61% <100.00%> (ø)

... and 280 files with indirect coverage changes

@bkoelman bkoelman marked this pull request as ready for review July 25, 2023 20:34
@bkoelman bkoelman merged commit e9c1e9b into openapi Jul 25, 2023
@bkoelman bkoelman deleted the merge-master-v5.3.0-into-openapi branch July 25, 2023 20:34
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.

2 participants