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 MissingGreenlet in getting new test tasks #866

Merged
merged 1 commit into from
Jun 12, 2024

Conversation

Korulag
Copy link
Contributor

@Korulag Korulag commented Jun 12, 2024

The issue with linked builds is that we didn't ask for linked builds' build repositories, thus when we tried process the build with linked builds we would fail each time.

The issue with linked builds is that we didn't ask for linked builds'
build repositories, thus when we tried process the build with linked
builds we would fail each time.
Copy link

73 passed, 11 skipped

Code Coverage Summary

Package Line Rate
alws 76%
alws.auth 77%
alws.auth.oauth 100%
alws.crud 41%
alws.dramatiq 60%
alws.middlewares 93%
alws.perms 86%
alws.routers 57%
alws.schemas 80%
alws.utils 41%
Summary 57% (5671 / 10009)

Linter reports

Pylint report
************* Module alws.crud.test
alws/crud/test.py:66:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/test.py:210:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/test.py:336:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/test.py:384:8: W1203: Use lazy % formatting in logging functions (logging-fstring-interpolation)
alws/crud/test.py:387:8: W1203: Use lazy % formatting in logging functions (logging-fstring-interpolation)
alws/crud/test.py:422:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/test.py:422:0: R0914: Too many local variables (17/15) (too-many-locals)
alws/crud/test.py:476:11: W0718: Catching too general exception Exception (broad-exception-caught)
alws/crud/test.py:460:4: R1702: Too many nested blocks (6/5) (too-many-nested-blocks)
alws/crud/test.py:422:0: R0912: Too many branches (17/12) (too-many-branches)

-----------------------------------
Your code has been rated at 9.55/10


Bandit report
Run started:2024-06-12 09:29:26.752071

Test results:
	No issues identified.

Code scanned:
	Total lines of code: 528
	Total lines skipped (#nosec): 0
	Total potential issues skipped due to specifically being disabled (e.g., #nosec BXXX): 0

Run metrics:
	Total issues (by severity):
		Undefined: 0
		Low: 0
		Medium: 0
		High: 0
	Total issues (by confidence):
		Undefined: 0
		Low: 0
		Medium: 0
		High: 0
Files skipped (0):

View full reports on the Job Summary page.

@Korulag Korulag merged commit e842886 into master Jun 12, 2024
2 of 3 checks passed
@Korulag Korulag deleted the fix-tests-linked-builds branch June 12, 2024 09:31
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