Add CodeQL to CI pipeline #63
Conversation
Codecov ReportBase: 97.71% // Head: 97.86% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #63 +/- ##
==========================================
+ Coverage 97.71% 97.86% +0.14%
==========================================
Files 16 17 +1
Lines 700 701 +1
==========================================
+ Hits 684 686 +2
+ Misses 16 15 -1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
|
This pull request fixes 3 alerts when merging 189ee86 into fad30ad - view on LGTM.com fixed alerts:
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog. |
codingjoe
force-pushed
the
codeql
branch
2 times, most recently
from
cf4eaab to
237fc0f
Compare
|
This pull request introduces 1 alert and fixes 3 when merging 237fc0f into fad30ad - view on LGTM.com new alerts:
fixed alerts:
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog. |
codingjoe
force-pushed
the
codeql
branch
2 times, most recently
from
55f34de to
4cf76ee
Compare
|
This pull request introduces 2 alerts and fixes 3 when merging 4cf76ee into fad30ad - view on LGTM.com new alerts:
fixed alerts:
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog. |
|
This pull request introduces 2 alerts and fixes 3 when merging 4129172 into fad30ad - view on LGTM.com new alerts:
fixed alerts:
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog. |
| return "https://github.com/%s/%s/blob/%s/%s.py%s" % ( | ||
| github_user, | ||
| project, | ||
| head, |
Check notice
Code scanning / CodeQL
Empty except
|
|
||
| def get_workflow(name) -> typing.Optional[Workflow]: | ||
| for workflow_cls in get_workflows(): | ||
| if ( |
Check notice
Code scanning / CodeQL
Explicit returns mixed with implicit (fall through) returns
| import traceback | ||
| from typing import Any, List, Tuple | ||
| import types | ||
| import typing |
Check notice
Code scanning / CodeQL
Module is imported with 'import' and 'import from'
| A task can be considered as a simple transaction that changes state of a workflow. | ||
| There are two types of tasks, human and machine tasks. | ||
| """ | ||
| from joeflow.typing import * # NoQA |
Check notice
Code scanning / CodeQL
'import *' may pollute namespace
|
This pull request introduces 2 alerts and fixes 3 when merging e3a3fe5 into fad30ad - view on LGTM.com new alerts:
fixed alerts:
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog. |
No description provided.