Skip to content
This repository has been archived by the owner on Mar 21, 2019. It is now read-only.

Add Flake8 support for tox test runner #13

Merged
merged 10 commits into from
Dec 22, 2017
Merged

Commits on Dec 14, 2017

  1. Configuration menu
    Copy the full SHA
    02cfd77 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    4e40234 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    3e40786 View commit details
    Browse the repository at this point in the history
  4. add flake8 testenv to tox

    Add a simple testenv which runs flake8 on our source
    rjw57 committed Dec 14, 2017
    Configuration menu
    Copy the full SHA
    b1fe075 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    29c20bd View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    e971fce View commit details
    Browse the repository at this point in the history
  7. doc: add note on flake8 tests

    rjw57 committed Dec 14, 2017
    Configuration menu
    Copy the full SHA
    e2e3095 View commit details
    Browse the repository at this point in the history
  8. Configuration menu
    Copy the full SHA
    aeb3681 View commit details
    Browse the repository at this point in the history

Commits on Dec 15, 2017

  1. set maximum line length to 99

    PEP 8 has the following to say about the subject of line length:
    
    "Some teams strongly prefer a longer line length. For code maintained
    exclusively or primarily by a team that can reach agreement on this
    issue, it is okay to increase the nominal line length from 80 to 100
    characters (effectively increasing the maximum length to 99 characters),
    provided that comments and docstrings are still wrapped at 72
    characters."
    
    As discussed in uisautomation#13, we've settled on 99 as a compromise between
    sticking to PEP8 and making use of modern screen widths.
    
    Update the editorconfig and add a flake8 configuration file to the root
    of the repo.
    rjw57 committed Dec 15, 2017
    Configuration menu
    Copy the full SHA
    7f2f756 View commit details
    Browse the repository at this point in the history
  2. settings.py: remove unnecessary flake8 silencing

    The line length silencing is now no-longer required because we allow a
    longer line length.
    rjw57 committed Dec 15, 2017
    Configuration menu
    Copy the full SHA
    f7d29fd View commit details
    Browse the repository at this point in the history