[Buildbot-commits] [buildbot/buildbot] 960835: Add Makefile target for running pep8 and pep8 conf...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/buildbot/buildbot
  Commit: 960835edd77cbe46aded2e03a87c81364c6edbb1
      https://github.com/buildbot/buildbot/commit/960835edd77cbe46aded2e03a87c81364c6edbb1
  Author: Vladimir Rutsky <altsysrq at gmail.com>
  Date:   2013-10-18 (Fri, 18 Oct 2013)

  Changed paths:
    M Makefile
    A common/pep8rc

  Log Message:
  -----------
  Add Makefile target for running pep8 and pep8 configuration

Currently pep8 configured to ignore ALL errors that occures in Buildbot
code.


  Commit: d0043f3171c2b33115fcc8e6ddc95d469b760f88
      https://github.com/buildbot/buildbot/commit/d0043f3171c2b33115fcc8e6ddc95d469b760f88
  Author: Vladimir Rutsky <altsysrq at gmail.com>
  Date:   2013-10-18 (Fri, 18 Oct 2013)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  Run pep8 on Travis


  Commit: ba14162482d03758c198b3da02d63fad5f100a7a
      https://github.com/buildbot/buildbot/commit/ba14162482d03758c198b3da02d63fad5f100a7a
  Author: Vladimir Rutsky <altsysrq at gmail.com>
  Date:   2013-10-18 (Fri, 18 Oct 2013)

  Changed paths:
    M common/pep8rc
    M master/buildbot/status/web/base.py
    M master/buildbot/test/unit/test_changes_p4poller.py
    M master/buildbot/test/unit/test_steps_package_rpm_rpmbuild.py

  Log Message:
  -----------
  Fix pep8 issue E101: indentation contains mixed spaces and tabs


  Commit: 848bec3b85fdf7a4e2fa099d61ff06f2b5c01310
      https://github.com/buildbot/buildbot/commit/848bec3b85fdf7a4e2fa099d61ff06f2b5c01310
  Author: Vladimir Rutsky <altsysrq at gmail.com>
  Date:   2013-10-18 (Fri, 18 Oct 2013)

  Changed paths:
    M Makefile

  Log Message:
  -----------
  Check only master and slave subdirectories with pep8

www/ subdirectory contains third party Python scripts which coding style
is not interesting to us.


  Commit: 85800580d478b73da2b24d7f6a6d512e245a0f57
      https://github.com/buildbot/buildbot/commit/85800580d478b73da2b24d7f6a6d512e245a0f57
  Author: Vladimir Rutsky <altsysrq at gmail.com>
  Date:   2013-10-18 (Fri, 18 Oct 2013)

  Changed paths:
    M common/pep8rc
    M master/buildbot/changes/changes.py
    M master/buildbot/changes/gerritchangesource.py
    M master/buildbot/changes/mail.py
    M master/buildbot/db/buildrequests.py
    M master/buildbot/db/buildsets.py
    M master/buildbot/process/properties.py
    M master/buildbot/schedulers/forcesched.py
    M master/buildbot/scripts/stop.py
    M master/buildbot/sourcestamp.py
    M master/buildbot/status/web/builder.py
    M master/buildbot/test/fake/web.py
    M master/buildbot/test/unit/test_db_buildrequests.py
    M master/buildbot/test/unit/test_db_buildsets.py
    M master/buildbot/test/unit/test_sourcestamp.py
    M master/buildbot/test/unit/test_status_mail.py
    M master/contrib/SimpleConfig.py
    M master/contrib/buildbot_cvs_mail.py
    M slave/buildslave/commands/repo.py
    M slave/contrib/windows/buildbot_service.py

  Log Message:
  -----------
  Fix pep8 error E111: indentation is not a multiple of four

File master/contrib/buildbot_json.py was completely excluded from pep8
analysis because it uses two spaces for indentation.


  Commit: 5118be2ae9b54f2095a4c719a7e2da3fe28eaba6
      https://github.com/buildbot/buildbot/commit/5118be2ae9b54f2095a4c719a7e2da3fe28eaba6
  Author: Dustin J. Mitchell <dustin at mozilla.com>
  Date:   2013-10-19 (Sat, 19 Oct 2013)

  Changed paths:
    M .travis.yml
    M Makefile
    A common/pep8rc
    M master/buildbot/changes/changes.py
    M master/buildbot/changes/gerritchangesource.py
    M master/buildbot/changes/mail.py
    M master/buildbot/db/buildrequests.py
    M master/buildbot/db/buildsets.py
    M master/buildbot/process/properties.py
    M master/buildbot/schedulers/forcesched.py
    M master/buildbot/scripts/stop.py
    M master/buildbot/sourcestamp.py
    M master/buildbot/status/web/base.py
    M master/buildbot/status/web/builder.py
    M master/buildbot/test/fake/web.py
    M master/buildbot/test/unit/test_changes_p4poller.py
    M master/buildbot/test/unit/test_db_buildrequests.py
    M master/buildbot/test/unit/test_db_buildsets.py
    M master/buildbot/test/unit/test_sourcestamp.py
    M master/buildbot/test/unit/test_status_mail.py
    M master/buildbot/test/unit/test_steps_package_rpm_rpmbuild.py
    M master/contrib/SimpleConfig.py
    M master/contrib/buildbot_cvs_mail.py
    M slave/buildslave/commands/repo.py
    M slave/contrib/windows/buildbot_service.py

  Log Message:
  -----------
  Merge branch 'use_pep8' of git://github.com/vrutsky/buildbot


  Commit: 9dbcb9f43b6470e78b90e596747a3cb4da603d6d
      https://github.com/buildbot/buildbot/commit/9dbcb9f43b6470e78b90e596747a3cb4da603d6d
  Author: Dustin J. Mitchell <dustin at mozilla.com>
  Date:   2013-10-19 (Sat, 19 Oct 2013)

  Changed paths:
    M common/validate.sh

  Log Message:
  -----------
  add pep8 to validate


Compare: https://github.com/buildbot/buildbot/compare/8efbee5b886a...9dbcb9f43b64