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

Fix GMR code generation #117

Merged
merged 10 commits into from
Feb 5, 2025
Merged

Fix GMR code generation #117

merged 10 commits into from
Feb 5, 2025

Conversation

tjpeel-ee
Copy link
Contributor

This PR presents no operational change, it is purely fixing the code generation in order to get us to a stable position.

Thanks to @t11omas for your time and input in addressing some of the issues.

We accept the invalid formatting as that is what the code generation gives us.

Missing string[] properties are now included, which were previously missing.

The CLI tool (for GMR generation only) now runs from the repository root and does not have fixed paths.

Copy link

sonarqubecloud bot commented Feb 5, 2025

Copy link

github-actions bot commented Feb 5, 2025

Code Coverage

Package Line Rate Branch Rate Health
Btms.Consumers 85% 77%
Btms.Azure 43% 100%
Btms.Business 82% 74%
Btms.Types.Alvs.Mapping.V1.Tests 0% 0%
Btms.Types.Ipaffs 95% 62%
Btms.Types.Gvms 61% 100%
Btms.Types.Alvs.Mapping 81% 0%
Btms.Backend 62% 39%
Btms.Types.Gvms.Mapping 35% 22%
Btms.Model 91% 94%
Btms.Common 74% 56%
TestGenerator.IntegrationTesting.Backend 83% 71%
TestDataGenerator 86% 80%
Btms.Types.Ipaffs.Mapping 77% 50%
Btms.BlobService 39% 27%
Btms.Metrics 84% 0%
Btms.Backend.Data 64% 58%
Btms.Analytics 74% 73%
Btms.Types.Alvs 85% 67%
Btms.SensitiveData 79% 83%
Btms.SyncJob 63% 45%
Btms.Emf 11% 0%
Summary 78% (9992 / 12763) 62% (1005 / 1619)

@tjpeel-ee tjpeel-ee merged commit 5972f63 into main Feb 5, 2025
3 checks passed
@tjpeel-ee tjpeel-ee deleted the PHAIN-74-codegen branch February 5, 2025 11:58
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