Closed batisteo closed 3 years ago
Kudos, SonarCloud Quality Gate passed!
This pull request fixes 1 alert when merging e6c0ab9f02a96a3818a526954c37941db75e5df1 into 4fef134a0c16fb3c95578869d09bb0da2dedd08f - view on LGTM.com
fixed alerts:
Description | Example fixes |
---|---|
Array type using 'Array |
:wrench: Example fixes |
β Merging this PR will decrease code quality in the affected files by 0.41%.
Quality metrics | Before | After | Change |
---|---|---|---|
Complexity | 0.65 β | 0.65 β | 0.00 |
Method Length | 43.80 β | 44.05 β | 0.25 π |
Working memory | 11.65 π | 11.78 π | 0.13 π |
Quality | 72.34% π | 71.93% π | -0.41% π |
Other metrics | Before | After | Change |
---|---|---|---|
Lines | 655 | 658 | 3 |
Changed files | Quality Before | Quality After | Quality Change |
---|---|---|---|
test/api/query/test_node.py | 79.46% β | 79.46% β | 0.00% |
test/api/query/snapshots/snap_test_node.py | 55.16% π | 52.77% π | -2.39% π |
zoo/factories.py | 51.95% π | 51.49% π | -0.46% π |
zoo/api/types.py | 83.55% β | 82.86% β | -0.69% π |
zoo/base/management/commands/fake.py | 75.58% β | 75.58% β | 0.00% |
zoo/services/managers.py | 71.40% π | 72.01% π | 0.61% π |
Here are some functions in these files that still need a tune-up:
File | Function | Complexity | Length | Working Memory | Quality | Recommendation |
---|---|---|---|---|---|---|
zoo/services/managers.py | SentryIssueQuerySet.generate_sentry_histogram | 5 β | 96 π | 15 π | 56.72% π | Extract out complex expressions |
zoo/api/types.py | Dependency.resolve_all_dependency_usages | 1 β | 92 π | 11 π | 67.22% π | Extract out complex expressions |
zoo/api/types.py | PagerdutyInfo.resolve_all_active_incidents | 1 β | 75 π | 11 π | 69.70% π | Extract out complex expressions |
test/api/query/test_node.py | test_issue | 0 β | 50 β | 14 π | 69.87% π | Extract out complex expressions |
zoo/api/types.py | Service.from_db | 0 β | 40 β | 12 π | 74.47% π | Extract out complex expressions |
The emojis denote the absolute quality of the code:
The π and π indicate whether the quality has improved or gotten worse with this pull request.
Please see our documentation here for details on how these metrics are calculated.
We are actively working on this report - lots more documentation and extra metrics to come!
Let us know what you think of it by mentioning @sourcery-ai in a comment.
This pull request fixes 1 alert when merging c609584ab0ce72bee0c713274db7d64db0f6bde9 into 4fef134a0c16fb3c95578869d09bb0da2dedd08f - view on LGTM.com
fixed alerts:
This pull request introduces 1 alert and fixes 1 when merging 11aeeff4b60af45724c4bfcd1d497bee4419161e into 4fef134a0c16fb3c95578869d09bb0da2dedd08f - view on LGTM.com
new alerts:
fixed alerts: