<p dir="auto">Scrapy docs recommends using PEP8 coding convention when writing code for inclusion in Scrapy, with the exception of 79 character line length rule . A quick run of <code class="notranslate">pep8</code> utility shows me that there are several occurrence of PEP8 violation in the master branch. If given the nod from the maintainers, I would like to work on resolving this.</p>
<p dir="auto">PEP8 is <a href="http://doc.scrapy.org/en/latest/contributing.html#coding-style" rel="nofollow">recommended in the docs</a> but is not strictly enforced.</p>
<p dir="auto">Some violations:</p>
<ol dir="auto">
<li>Multiple imports in <a href="https://github.com/scrapy/scrapy/blob/master/scrapy/commands/edit.py#L1">one line</a></li>
<li>Two blank lines <a href="https://github.com/scrapy/scrapy/blob/master/scrapy/commands/edit.py#L5">not maintained</a> between class definitions.</li>
<li><a href="https://github.com/scrapy/scrapy/blob/master/scrapy/commands/check.py#L3">Unused imports</a></li>
<li>Lines longer than 80 characters everywhere</li>
</ol>
<p dir="auto">I believe we should strictly enforce PEP8 and also edit our CI configurations to test for these violations.<br>
I would love to submit a patch for this if the idea sounds good to everyone.</p>