Sign upLogin

clemfromspace/scrapy-selenium

View on GitHub
Star

Showing 3 of 5 total issues

Function __init__ has 6 arguments (exceeds 4 allowed). Consider refactoring.
Open

    def __init__(self, wait_time=None, wait_until=None, screenshot=False, script=None, *args, **kwargs):
Severity: Minor
Found in scrapy_selenium/http.py - About 45 mins to fix

    Blank line at end of file
    Open

    Severity: Minor
    Found in setup.py by pep8

    Trailing blank lines are superfluous.

    Okay: spam(1)
W391: spam(1)\n

However the last line should end with a new line (warning W292).

    Line too long (104 > 100 characters)
    Open

        def __init__(self, wait_time=None, wait_until=None, screenshot=False, script=None, *args, **kwargs):
    Severity: Minor
    Found in scrapy_selenium/http.py by pep8

    Limit all lines to a maximum of 79 characters.

    There are still many devices around that are limited to 80 character
lines; plus, limiting windows to 80 characters makes it possible to
have several windows side-by-side.  The default wrapping on such
devices looks ugly.  Therefore, please limit all lines to a maximum
of 79 characters. For flowing long blocks of text (docstrings or
comments), limiting the length to 72 characters is recommended.

Reports error E501.
    Severity
    Category
    Status
    Source
    Language