kiwicom / the-zoo

πŸ’ A service registry for humans
MIT License
92 stars 6 forks source link

feat: customize graphql for resources frontend #366

Closed gjuro87 closed 3 years ago

gjuro87 commented 3 years ago

Everything is done. There is one issue that i solved in another way. Issue when calling dependency graphql query and requesting id. Result is error: "'Dependency' object has no attribute 'pk'". I couldn't find the cause. I have solve it another way, not requesting id, but needs to be solved. If you could take a look

lgtm-com[bot] commented 3 years ago

This pull request fixes 5 alerts when merging 0868fde2adc1df4d2bb4d1b376f1329136333416 into 8a83302f0903821f316cf7c4249defb11ac6f90c - view on LGTM.com

fixed alerts:

gjuro87 commented 3 years ago

Bug fixed, thx for help

lgtm-com[bot] commented 3 years ago

This pull request fixes 5 alerts when merging 0ba4e4468708d7b4854fb185b8e5c9a49b33a539 into 8a83302f0903821f316cf7c4249defb11ac6f90c - view on LGTM.com

fixed alerts:

batisteo commented 3 years ago

For the Pytest check, run pytest test/api/query/test_node.py --snapshot-update and review https://github.com/kiwicom/the-zoo/blob/hangar/test/api/query/snapshots/snap_test_node.py

lgtm-com[bot] commented 3 years ago

This pull request fixes 5 alerts when merging e7c7cf0eacad8f3cdb3f58291c4213954210d1ab into 8a83302f0903821f316cf7c4249defb11ac6f90c - view on LGTM.com

fixed alerts:

lgtm-com[bot] commented 3 years ago

This pull request fixes 5 alerts when merging 663e2f9b1cfcc4b85307461c66193f00d398db7e into 8a83302f0903821f316cf7c4249defb11ac6f90c - view on LGTM.com

fixed alerts:

ghost commented 3 years ago

Congratulations :tada:. DeepCode analyzed your code in 6.281 seconds and we found no issues. Enjoy a moment of no bugs :sunny:.

πŸ‘‰ View analysis in DeepCode’s Dashboard | Configure the bot

sourcery-ai[bot] commented 3 years ago

Sourcery Code Quality Report

❌  Merging this PR will decrease code quality in the affected files by 0.97%.

Quality metrics Before After Change
Complexity 0.45 ⭐ 0.38 ⭐ -0.07 πŸ‘
Method Length 35.63 ⭐ 36.68 ⭐ 1.05 πŸ‘Ž
Working memory 7.75 πŸ™‚ 7.78 πŸ™‚ 0.03 πŸ‘Ž
Quality 83.66% ⭐ 82.69% ⭐ -0.97% πŸ‘Ž
Other metrics Before After Change
Lines 408 396 -12
Changed files Quality Before Quality After Quality Change
test/api/query/snapshots/snap_test_node.py 46.29% 😞 47.79% 😞 1.50% πŸ‘
zoo/api/types.py 85.16% ⭐ 84.07% ⭐ -1.09% πŸ‘Ž

Here are some functions in these files that still need a tune-up:

File Function Complexity Length Working Memory Quality Recommendation
zoo/api/types.py SentryIssue.resolve_histogram 0 ⭐ 89 πŸ™‚ 18 β›” 59.69% πŸ™‚ 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
zoo/api/types.py Service.resolve_all_environments 1 ⭐ 83 πŸ™‚ 10 😞 70.41% πŸ™‚ Extract out complex expressions
zoo/api/types.py Service.from_db 0 ⭐ 40 ⭐ 12 😞 74.47% πŸ™‚ Extract out complex expressions

Legend and Explanation

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.

lgtm-com[bot] commented 3 years ago

This pull request fixes 5 alerts when merging 8df53cd2b48b78449c94cb56c05329e1cee2f3fb into 8a83302f0903821f316cf7c4249defb11ac6f90c - view on LGTM.com

fixed alerts:

sonarcloud[bot] commented 3 years ago

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

89.5% 89.5% Coverage
0.0% 0.0% Duplication