[Pytest-commit] [pytest-dev/pytest] 3862b0: Remove logger parameter from catching_logs

Bruno Oliveira nicoddemus at gmail.com
Wed Dec 13 17:56:50 EST 2017


  Branch: refs/heads/master
  Home:   https://github.com/pytest-dev/pytest
  Commit: 3862b0b28d1dac1d9ea398fc6155ed5e6de5498c
      https://github.com/pytest-dev/pytest/commit/3862b0b28d1dac1d9ea398fc6155ed5e6de5498c
  Author: Thomas Hisch <t.hisch at gmail.com>
  Date:   2017-12-13 (Wed, 13 Dec 2017)

  Changed paths:
    M _pytest/logging.py

  Log Message:
  -----------
  Remove logger parameter from catching_logs

The logger parameter of catching_logs is not used anywhere. The main
motivation for removing the logger parameter is that it removes the

logger = logger or logging.getLogger(logger)

line. IMO there are too many occurences of the string 'logg' ;)


  Commit: b17c6e5f8975fd02405c7eeac10ad2183110569f
      https://github.com/pytest-dev/pytest/commit/b17c6e5f8975fd02405c7eeac10ad2183110569f
  Author: Bruno Oliveira <nicoddemus at gmail.com>
  Date:   2017-12-13 (Wed, 13 Dec 2017)

  Changed paths:
    M _pytest/logging.py

  Log Message:
  -----------
  Merge pull request #3028 from thisch/removelogger

Remove logger parameter from catching_logs


Compare: https://github.com/pytest-dev/pytest/compare/476d4df1b7e0...b17c6e5f8975


More information about the pytest-commit mailing list