The use of else or elif becomes redundant and can be dropped if the last statement under the leading if / elif block is a return statement. In the case of an elif after return, it can be written as a separate if block. For else blocks after return, the …
Occurrences
There is 1 occurrence of this issue in the repository.
Defining a local variable or function with the same name as a built-in object makes the built-in object unusable within the current scope and makes the code prone to bugs.
Occurrences
There are 4 occurrences of this issue in the repository.
Formatting the message manually before passing it to a logging call does unnecessary work if logging in disabled. Consider using the logging module's built-in formatting features to avoid that. <!-- more -->
Occurrences
There are 7 occurrences of this issue in the repository.
Hey there,
first: thanks for the bot :) i would like to know, is it possible to search just for FN for example, also what does positive_regex and negetive do?
Thanks Thomas
An unused variable takes up space in the code, and can lead to confusion, and it should be removed. If this variable is necessary, name the variable _ to indicate that it will be unused, or start the name with unused or _unused.
Occurrences
There is 1 occurrence of this issue in the repository.
The module/function docstring is missing. If you want to ignore this, you can configure this in the .deepsource.toml file. Please refer to docs for available options.
Occurrences
There are 2 occurrences of this issue in the repository.