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

[GitHub]

  Branch: refs/heads/nine
  Home:   https://github.com/buildbot/buildbot
  Commit: e6b90f6a40891082785ed049d6984bce648a5caa
      https://github.com/buildbot/buildbot/commit/e6b90f6a40891082785ed049d6984bce648a5caa
  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: d823574a52a7561266c6aa41e5896c6bd3f877d8
      https://github.com/buildbot/buildbot/commit/d823574a52a7561266c6aa41e5896c6bd3f877d8
  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: 9a233aeaa3beb3af6216a7a1bb25892e6bb530e0
      https://github.com/buildbot/buildbot/commit/9a233aeaa3beb3af6216a7a1bb25892e6bb530e0
  Author: Vladimir Rutsky <altsysrq at gmail.com>
  Date:   2013-10-18 (Fri, 18 Oct 2013)

  Changed paths:
    M common/pep8rc
    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: 4102d8f3f1f7bf504e4771382356c844c965bfd7
      https://github.com/buildbot/buildbot/commit/4102d8f3f1f7bf504e4771382356c844c965bfd7
  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: 37efd76b3879a5fe8cb12201421256280b48a678
      https://github.com/buildbot/buildbot/commit/37efd76b3879a5fe8cb12201421256280b48a678
  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/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_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: 9507d33fc078d8d4a3d50895c56b52db7ba39144
      https://github.com/buildbot/buildbot/commit/9507d33fc078d8d4a3d50895c56b52db7ba39144
  Author: Vladimir Rutsky <altsysrq at gmail.com>
  Date:   2013-10-18 (Fri, 18 Oct 2013)

  Changed paths:
    M master/buildbot/test/unit/test_changes_p4poller.py

  Log Message:
  -----------
  Fix buildbot.test.unit.test_changes_p4poller tests

Not sure is strings format is critical.  Is P4 have fixed output format
with mandatory tabs?


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

  Changed paths:
    M master/buildbot/db/changesources.py
    M master/buildbot/db/schedulers.py
    M master/buildbot/process/buildstep.py

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


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

  Changed paths:
    M Makefile

  Log Message:
  -----------
  Check with pep8 www subdirectory


  Commit: c14c93cb86f4b722314b6d2067c81df404700bfd
      https://github.com/buildbot/buildbot/commit/c14c93cb86f4b722314b6d2067c81df404700bfd
  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/db/changesources.py
    M master/buildbot/db/schedulers.py
    M master/buildbot/process/buildstep.py
    M master/buildbot/process/properties.py
    M master/buildbot/schedulers/forcesched.py
    M master/buildbot/scripts/stop.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_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 'nine_use_pep8' of git://github.com/vrutsky/buildbot into nine


Compare: https://github.com/buildbot/buildbot/compare/db32066dbdd0...c14c93cb86f4