Skip to content

Security analysis#309

Open
Yarauni wants to merge 27 commits intoishepard:masterfrom
Yarauni:security-analysis
Open

Security analysis#309
Yarauni wants to merge 27 commits intoishepard:masterfrom
Yarauni:security-analysis

Conversation

@Yarauni
Copy link

@Yarauni Yarauni commented Mar 10, 2025

No description provided.

if line.startswith('__version__'):
delim = '"' if '"' in line else "'"
return line.split(delim)[1]
except FileNotFoundError:

Check notice

Code scanning / CodeQL

Empty except Note

'except' clause does nothing but pass and there is no explanatory comment.


import os
import pytest

Check notice

Code scanning / CodeQL

Unused import Note test

Import of 'pytest' is not used.
return "Low"


def extract_security_diffs_and_store(repo_url, since=None, to=None, continuous=False, interval=300, max_workers=4):

Check notice

Code scanning / CodeQL

Explicit returns mixed with implicit (fall through) returns Note test

Mixing implicit and explicit returns may indicate an error as implicit returns always return None.
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.

1 participant