-
Notifications
You must be signed in to change notification settings - Fork 73
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
19374 & 19375 amalgamation validaton fixes #2397
19374 & 19375 amalgamation validaton fixes #2397
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #2397 +/- ##
==========================================
- Coverage 77.37% 76.76% -0.61%
==========================================
Files 202 168 -34
Lines 11602 10523 -1079
Branches 1961 1789 -172
==========================================
- Hits 8977 8078 -899
+ Misses 2043 1959 -84
+ Partials 582 486 -96
Flags with carried forward coverage won't be shown. Click here to find out more.
|
legal-api/src/legal_api/services/filings/validations/amalgamation_application.py
Outdated
Show resolved
Hide resolved
legal-api/src/legal_api/services/filings/validations/amalgamation_application.py
Outdated
Show resolved
Hide resolved
legal-api/src/legal_api/services/filings/validations/amalgamation_application.py
Show resolved
Hide resolved
Quality Gate failedFailed conditions 17.9% Duplication on New Code (required ≤ 3%) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Issue #: /bcgov/entity#19374 and /bcgov/entity#19375
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the lear license (Apache 2.0).