Skip to content

Merge master into openapi #1339

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 45 commits into from
Sep 19, 2023
Merged

Merge master into openapi #1339

merged 45 commits into from
Sep 19, 2023

Conversation

bkoelman
Copy link
Member

No description provided.

bkoelman and others added 30 commits August 17, 2023 10:50
* Update to Resharper v2023.2.1

* Add workaround for Resharper bug at https://youtrack.jetbrains.com/issue/RSRP-493256/Incorrect-possible-null-assignment

* Use new formatting directives

* Fail the build when capturing closures in lambdas within source generators project (they cause expensive allocations)
Add example for scopes-based authorization
…nstead of hang when webserver does not start
…age-version

GitHub Actions: Fix the missing branch name in NuGet package version on PR merge
… in favor of enumeration Forbidden/Allowed/Required
…r-type

Client-generated IDs per resource type
Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v3...v4)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@bkoelman bkoelman changed the base branch from master to openapi September 19, 2023 21:44
@bkoelman bkoelman force-pushed the merge-master-into-openapi branch from ed260a6 to d61e55d Compare September 19, 2023 21:52
@codecov
Copy link

codecov bot commented Sep 19, 2023

Codecov Report

Merging #1339 (43f4890) into openapi (e9c1e9b) will decrease coverage by 4.16%.
The diff coverage is 91.66%.

@@             Coverage Diff             @@
##           openapi    #1339      +/-   ##
===========================================
- Coverage    96.81%   92.66%   -4.16%     
===========================================
  Files           21      329     +308     
  Lines          597     9856    +9259     
===========================================
+ Hits           578     9133    +8555     
- Misses          19      723     +704     
Files Changed Coverage Δ
src/JsonApiDotNetCore.Annotations/ArgumentGuard.cs 80.00% <ø> (ø)
.../JsonApiDotNetCore.OpenApi.Client/ArgumentGuard.cs 100.00% <ø> (ø)
...tNetCore.OpenApi.Client/Exceptions/ApiException.cs 87.50% <ø> (ø)
...piDotNetCore.OpenApi/ActionDescriptorExtensions.cs 100.00% <ø> (ø)
...piDotNetCore.OpenApi/JsonApiOperationIdSelector.cs 92.45% <ø> (ø)
...onApiDotNetCore/AtomicOperations/LocalIdTracker.cs 97.50% <ø> (ø)
...ore/AtomicOperations/OperationProcessorAccessor.cs 95.23% <ø> (ø)
...DotNetCore/AtomicOperations/OperationsProcessor.cs 96.87% <ø> (ø)
...perations/Processors/AddToRelationshipProcessor.cs 100.00% <ø> (ø)
...ore/AtomicOperations/Processors/CreateProcessor.cs 100.00% <ø> (ø)
... and 60 more

... and 239 files with indirect coverage changes

@github-actions
Copy link

github-actions bot commented Sep 19, 2023

Qodana for .NET

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

@bkoelman bkoelman force-pushed the merge-master-into-openapi branch from 206b016 to 84f6cec Compare September 19, 2023 22:19
@bkoelman bkoelman marked this pull request as ready for review September 19, 2023 23:03
@bkoelman bkoelman merged commit 69e3a58 into openapi Sep 19, 2023
@bkoelman bkoelman deleted the merge-master-into-openapi branch September 19, 2023 23:03
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