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

Cdms-183 #2

Merged
merged 4 commits into from
Dec 6, 2024
Merged

Cdms-183 #2

merged 4 commits into from
Dec 6, 2024

Conversation

t11omas
Copy link
Contributor

@t11omas t11omas commented Dec 6, 2024

  1. Updated KeyDataPairsToDictionaryStringObjectJsonConverter to supprrt when data is null, and added some tests

  2. Updated audit log to use a double when ValueKind is a number to support real numbers, and added some tests

  3. Updated document to documents

  4. Fixed an issue with concurrency within Linking due to the etag being empty

Thomas Anderson added 3 commits December 6, 2024 06:32
…rt when data is null, and added some tests

2. Updated audit log to use a double when ValueKind is a number to support real numbers, and added some tests
3. Updated document to documents
4. Fixed an issue with concurrency within Linking due to the etag being empty
@t11omas t11omas marked this pull request as ready for review December 6, 2024 07:38
Copy link

github-actions bot commented Dec 6, 2024

Code Coverage

Package Line Rate Branch Rate Health
Btms.Business 81% 68%
Btms.Types.Gvms.Mapping 38% 26%
Btms.Metrics 82% 0%
Btms.Types.Gvms 67% 100%
Btms.Types.Alvs.Mapping 74% 0%
Btms.Types.Ipaffs.Mapping 73% 40%
TestDataGenerator 54% 47%
Btms.Azure 70% 75%
Btms.BlobService 31% 17%
Btms.Analytics 80% 65%
Btms.Backend.Data 65% 62%
Btms.Consumers 70% 38%
Btms.Common 79% 17%
Btms.Model 90% 92%
Btms.SyncJob 81% 71%
Btms.SensitiveData 80% 81%
Btms.Types.Ipaffs 94% 56%
Btms.Types.Alvs.Mapping.V1.Tests 0% 0%
Btms.Types.Alvs 83% 67%
Btms.Emf 11% 6%
Btms.Backend 57% 57%
Summary 73% (5348 / 7360) 47% (641 / 1366)

@craigedmunds craigedmunds merged commit c497790 into main Dec 6, 2024
1 check passed
@craigedmunds craigedmunds deleted the feature/cdms-183 branch December 6, 2024 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants