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

Remove Required constraints of EntityChange's EntityId. #17979

Merged
merged 2 commits into from
Nov 1, 2023
Merged

Conversation

maliming
Copy link
Member

@maliming maliming commented Oct 26, 2023

@maliming maliming marked this pull request as ready for review October 26, 2023 09:08
@codecov
Copy link

codecov bot commented Oct 26, 2023

Codecov Report

Merging #17979 (eb1959d) into dev (0542f71) will increase coverage by 0.00%.
Report is 10 commits behind head on dev.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##              dev   #17979   +/-   ##
=======================================
  Coverage   51.64%   51.64%           
=======================================
  Files        3052     3052           
  Lines       95464    95464           
  Branches     7613     7613           
=======================================
+ Hits        49301    49306    +5     
+ Misses      44648    44643    -5     
  Partials     1515     1515           
Files Coverage Δ
.../AbpAuditLoggingDbContextModelBuilderExtensions.cs 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@maliming maliming requested review from hikalkan and realLiangshiwei and removed request for hikalkan October 31, 2023 09:20
@realLiangshiwei realLiangshiwei merged commit a750f00 into dev Nov 1, 2023
6 checks passed
@realLiangshiwei realLiangshiwei deleted the EntityId branch November 1, 2023 01:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants