Closed epignatelli closed 1 year ago
************* Module helx
helx/__init__.py:1:0: C0114: Missing module docstring (missing-module-docstring)
************* Module helx.download
helx/download.py:93:0: C0301: Line too long (110/100) (line-too-long)
helx/download.py:1:0: C0114: Missing module docstring (missing-module-docstring)
helx/download.py:38:4: C0103: Variable name "r" doesn't conform to snake_case naming style (invalid-name)
helx/download.py:38:8: W3101: Missing timeout argument for method 'requests.get' can cause your program to hang indefinitely (missing-timeout)
helx/download.py:39:4: W1203: Use lazy % formatting in logging functions (logging-fstring-interpolation)
helx/download.py:40:33: C0103: Variable name "fd" doesn't conform to snake_case naming style (invalid-name)
helx/download.py:47:0: C0116: Missing function or method docstring (missing-function-docstring)
helx/download.py:81:4: W1203: Use lazy % formatting in logging functions (logging-fstring-interpolation)
helx/download.py:92:0: C0116: Missing function or method docstring (missing-function-docstring)
helx/download.py:128:4: W1203: Use lazy % formatting in logging functions (logging-fstring-interpolation)
helx/download.py:134:0: C0116: Missing function or method docstring (missing-function-docstring)
helx/download.py:145:4: W1203: Use lazy % formatting in logging functions (logging-fstring-interpolation)
helx/download.py:152:11: W0703: Catching too general exception Exception (broad-except)
helx/download.py:152:4: C0103: Variable name "e" doesn't conform to snake_case naming style (invalid-name)
helx/download.py:159:0: C0116: Missing function or method docstring (missing-function-docstring)
Report
======
90 statements analysed.
Statistics by type
------------------
+---------+-------+-----------+-----------+------------+---------+
|type |number |old number |difference |%documented |%badname |
+=========+=======+===========+===========+============+=========+
|module |2 |NC |NC |0.00 |0.00 |
+---------+-------+-----------+-----------+------------+---------+
|class |0 |NC |NC |0 |0 |
+---------+-------+-----------+-----------+------------+---------+
|method |0 |NC |NC |0 |0 |
+---------+-------+-----------+-----------+------------+---------+
|function |5 |NC |NC |20.00 |0.00 |
+---------+-------+-----------+-----------+------------+---------+
188 lines have been analyzed
Raw metrics
-----------
+----------+-------+------+---------+-----------+
|type |number |% |previous |difference |
+==========+=======+======+=========+===========+
|code |101 |53.72 |NC |NC |
+----------+-------+------+---------+-----------+
|docstring |11 |5.85 |NC |NC |
+----------+-------+------+---------+-----------+
|comment |43 |22.87 |NC |NC |
+----------+-------+------+---------+-----------+
|empty |33 |17.55 |NC |NC |
+----------+-------+------+---------+-----------+
Duplication
-----------
+-------------------------+------+---------+-----------+
| |now |previous |difference |
+=========================+======+=========+===========+
|nb duplicated lines |0 |NC |NC |
+-------------------------+------+---------+-----------+
|percent duplicated lines |0.000 |NC |NC |
+-------------------------+------+---------+-----------+
Messages by category
--------------------
+-----------+-------+---------+-----------+
|type |number |previous |difference |
+===========+=======+=========+===========+
|convention |10 |NC |NC |
+-----------+-------+---------+-----------+
|refactor |0 |NC |NC |
+-----------+-------+---------+-----------+
|warning |6 |NC |NC |
+-----------+-------+---------+-----------+
|error |0 |NC |NC |
+-----------+-------+---------+-----------+
Messages
--------
+------------------------------+------------+
|message id |occurrences |
+==============================+============+
|missing-function-docstring |4 |
+------------------------------+------------+
|logging-fstring-interpolation |4 |
+------------------------------+------------+
|invalid-name |3 |
+------------------------------+------------+
|missing-module-docstring |2 |
+------------------------------+------------+
|missing-timeout |1 |
+------------------------------+------------+
|line-too-long |1 |
+------------------------------+------------+
|broad-except |1 |
+------------------------------+------------+
-----------------------------------
Your code has been rated at 8.22/10
Closes #22
setuptools_scm
setup.py
in favour ofpyproject.toml
(PEP517)helx-download-extras
pylint
s thediff
between the changes in a PR and the base branch