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

Pylint reportes lot of errors #36

Open
iblancasa opened this issue Feb 14, 2017 · 2 comments
Open

Pylint reportes lot of errors #36

iblancasa opened this issue Feb 14, 2017 · 2 comments

Comments

@iblancasa
Copy link
Contributor

Pylint is reporting lot of errors related to code style

@iblancasa
Copy link
Contributor Author

Report

No config file found, using default configuration
************* Module logparser
I: 21, 0: Locally disabling undefined-all-variable (E0603) (locally-disabled)
************* Module logparser.logparser
I:186, 0: Locally disabling broad-except (W0703) (locally-disabled)
C: 67, 8: Invalid variable name "MAX_TIME_SEC" (invalid-name)
C:159,12: Invalid variable name "originalOutput" (invalid-name)
C:204, 8: Invalid variable name "DATE_REGEX" (invalid-name)
C:207, 8: Invalid variable name "SINGLE_DATE_REGEX" (invalid-name)
C:109,12: Invalid attribute name "onlyIf" (invalid-name)
************* Module logparser.logger
C: 63, 8: Invalid attribute name "_showColors" (invalid-name)
C: 40, 8: Invalid attribute name "_COLORS" (invalid-name)
C: 64, 8: Invalid attribute name "_formatDevice" (invalid-name)
C: 62, 8: Invalid attribute name "_ignorePackets" (invalid-name)
C: 66, 8: Invalid attribute name "_onlyIf" (invalid-name)
C:131, 8: Invalid attribute name "ignorePackets" (invalid-name)
C: 53, 8: Invalid attribute name "_KIND_TO_COLOR" (invalid-name)
R: 22, 0: Too many instance attributes (16/7) (too-many-instance-attributes)
C:105, 4: Invalid attribute name "ignorePackets" (invalid-name)
C:114, 4: Invalid attribute name "ignorePackets" (invalid-name)
C:159, 4: Invalid attribute name "onlyIf" (invalid-name)
C:168, 4: Invalid attribute name "onlyIf" (invalid-name)
W:219,24: Used builtin function 'filter'. Using a list comprehension can be clearer. (bad-builtin)
************* Module logparser.utils
C:172, 4: Invalid variable name "BUILTIN_NAMES" (invalid-name)
C:186, 4: Invalid variable name "ENTITY_ORIGINS" (invalid-name)
C:187, 4: Invalid variable name "ENTITY_KINDS" (invalid-name)
C:220, 4: Invalid variable name "PACKET_NAMES" (invalid-name)
C:389, 4: Invalid variable name "FLAGS" (invalid-name)
C:400, 0: Invalid argument name "ip" (invalid-name)
C:440, 4: Invalid variable name "TRANSPORTS" (invalid-name)
************* Module logparser.logs.logs
I: 38, 0: Locally disabling expression-not-assigned (W0106) (locally-disabled)
************* Module logparser.logs.__init__
I: 18, 0: Locally disabling undefined-all-variable (E0603) (locally-disabled)
************* Module logparser.logs.events.events
I: 63, 0: Locally disabling unused-argument (W0613) (locally-disabled)
C: 91, 4: Invalid variable name "ip" (invalid-name)
C:398, 0: Invalid function name "on_announce_local_publication_sed" (invalid-name)
C:412, 0: Invalid function name "on_announce_local_subscription_sed" (invalid-name)
C:474, 0: Invalid function name "on_reader_incompatible_durability" (invalid-name)
C:476, 4: Invalid variable name "DURABILITY" (invalid-name)
************* Module logparser.logs.events.logs
R: 25, 0: Too many statements (70/50) (too-many-statements)
************* Module logparser.logs.events.__init__
I: 18, 0: Locally disabling undefined-all-variable (E0603) (locally-disabled)
************* Module logparser.logs.network.network
I: 64, 0: Locally disabling unused-argument (W0613) (locally-disabled)
C:128, 0: Invalid function name "on_unregister_not_asserted_entity" (invalid-name)
C:130, 4: Invalid function name "on_unregister_given_not_asserted_entity" (invalid-name)
C:140, 0: Invalid function name "on_send_participant_announcement" (invalid-name)
C:348, 0: Invalid function name "on_drop_unregister_no_ack_instance" (invalid-name)
C:358, 0: Invalid function name "on_writer_batching_exceed_max_entries" (invalid-name)
C:368, 0: Invalid function name "on_write_max_blocking_time_expired" (invalid-name)
C:537, 0: Invalid function name "on_sample_received_from_deleted_writer" (invalid-name)
************* Module logparser.logs.network.__init__
I: 18, 0: Locally disabling undefined-all-variable (E0603) (locally-disabled)
************* Module logparser.logs.routing.routing
I: 28, 0: Locally disabling unused-argument (W0613) (locally-disabled)
************* Module logparser.logs.routing.__init__
I: 18, 0: Locally disabling undefined-all-variable (E0603) (locally-disabled)
************* Module logparser.logs.debug.logs
R: 25, 0: Too many statements (105/50) (too-many-statements)
************* Module logparser.logs.debug.__init__
I: 18, 0: Locally disabling undefined-all-variable (E0603) (locally-disabled)
************* Module logparser.logs.debug.debug
I: 24, 0: Locally disabling unused-argument (W0613) (locally-disabled)
************* Module logparser.logs.custom.custom
I: 25, 0: Locally disabling unused-argument (W0613) (locally-disabled)
************* Module logparser.logs.custom.__init__
I: 18, 0: Locally disabling undefined-all-variable (E0603) (locally-disabled)
************* Module logparser.devices.inputdevices
I: 92, 0: Locally disabling broad-except (W0703) (locally-disabled)
I:155, 0: Locally disabling broad-except (W0703) (locally-disabled)
C:123, 4: Invalid argument name "barLength" (invalid-name)
C:139, 8: Invalid variable name "filledLength" (invalid-name)
C:141, 8: Invalid variable name "barText" (invalid-name)
************* Module logparser.devices.__init__
I: 18, 0: Locally disabling undefined-all-variable (E0603) (locally-disabled)


Report
======
1569 statements analysed.

Statistics by type
------------------

+---------+-------+-----------+-----------+------------+---------+
|type     |number |old number |difference |%documented |%badname |
+=========+=======+===========+===========+============+=========+
|module   |27     |27         |=          |100.00      |0.00     |
+---------+-------+-----------+-----------+------------+---------+
|class    |11     |11         |=          |100.00      |0.00     |
+---------+-------+-----------+-----------+------------+---------+
|method   |73     |73         |=          |100.00      |0.00     |
+---------+-------+-----------+-----------+------------+---------+
|function |145    |145        |=          |100.00      |6.90     |
+---------+-------+-----------+-----------+------------+---------+



External dependencies
---------------------
::

    logparser (logparser.devices.markdownformatdevice,logparser.logparser)
      \-devices 
      | \-formatdevice (logparser.devices.markdownformatdevice)
      | \-inputdevices 
      | | \-InputConsoleDevice (logparser.logparser)
      | | \-InputFileDevice (logparser.logparser)
      | \-outputdevices 
      |   \-OutputConsoleDevice (logparser.logparser)
      |   \-OutputFileDevice (logparser.logparser)
      \-logs 
      | \-custom 
      | | \-custom (logparser.logs.custom.logs)
      | | \-logs (logparser.logs.logs)
      | \-debug 
      | | \-debug (logparser.logs.debug.logs)
      | | \-logs (logparser.logs.logs)
      | \-events 
      | | \-events (logparser.logs.events.logs)
      | | \-logs (logparser.logs.logs)
      | \-logs 
      | | \-create_regex_list (logparser.logparser)
      | \-network 
      | | \-logs (logparser.logs.logs)
      | | \-network (logparser.logs.network.logs)
      | \-routing 
      |   \-logs (logparser.logs.logs)
      |   \-routing (logparser.logs.routing.logs)
      \-utils (logparser.logs.network.network,logparser.logs.events.events)



Raw metrics
-----------

+----------+-------+------+---------+-----------+
|type      |number |%     |previous |difference |
+==========+=======+======+=========+===========+
|code      |1728   |43.45 |1728     |=          |
+----------+-------+------+---------+-----------+
|docstring |1179   |29.65 |1179     |=          |
+----------+-------+------+---------+-----------+
|comment   |557    |14.01 |557      |=          |
+----------+-------+------+---------+-----------+
|empty     |513    |12.90 |513      |=          |
+----------+-------+------+---------+-----------+



Duplication
-----------

+-------------------------+------+---------+-----------+
|                         |now   |previous |difference |
+=========================+======+=========+===========+
|nb duplicated lines      |0     |0        |=          |
+-------------------------+------+---------+-----------+
|percent duplicated lines |0.000 |0.000    |=          |
+-------------------------+------+---------+-----------+



Messages by category
--------------------

+-----------+-------+---------+-----------+
|type       |number |previous |difference |
+===========+=======+=========+===========+
|convention |38     |38       |=          |
+-----------+-------+---------+-----------+
|refactor   |3      |3        |=          |
+-----------+-------+---------+-----------+
|warning    |1      |1        |=          |
+-----------+-------+---------+-----------+
|error      |0      |0        |=          |
+-----------+-------+---------+-----------+



% errors / warnings by module
-----------------------------

+-------------------------------+------+--------+---------+-----------+
|module                         |error |warning |refactor |convention |
+===============================+======+========+=========+===========+
|logparser.logger               |0.00  |100.00  |33.33    |31.58      |
+-------------------------------+------+--------+---------+-----------+
|logparser.logs.events.logs     |0.00  |0.00    |33.33    |0.00       |
+-------------------------------+------+--------+---------+-----------+
|logparser.logs.debug.logs      |0.00  |0.00    |33.33    |0.00       |
+-------------------------------+------+--------+---------+-----------+
|logparser.utils                |0.00  |0.00    |0.00     |18.42      |
+-------------------------------+------+--------+---------+-----------+
|logparser.logs.network.network |0.00  |0.00    |0.00     |18.42      |
+-------------------------------+------+--------+---------+-----------+
|logparser.logs.events.events   |0.00  |0.00    |0.00     |13.16      |
+-------------------------------+------+--------+---------+-----------+
|logparser.logparser            |0.00  |0.00    |0.00     |10.53      |
+-------------------------------+------+--------+---------+-----------+
|logparser.devices.inputdevices |0.00  |0.00    |0.00     |7.89       |
+-------------------------------+------+--------+---------+-----------+



Messages
--------

+-----------------------------+------------+
|message id                   |occurrences |
+=============================+============+
|invalid-name                 |38          |
+-----------------------------+------------+
|locally-disabled             |17          |
+-----------------------------+------------+
|too-many-statements          |2           |
+-----------------------------+------------+
|too-many-instance-attributes |1           |
+-----------------------------+------------+
|bad-builtin                  |1           |
+-----------------------------+------------+



Global evaluation
-----------------
Your code has been rated at 9.73/10 (previous run: 9.73/10, +0.00)

@iblancasa
Copy link
Contributor Author

Some of then are not correct erros

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant