Remove logger parameter from catching_logs

[twmr]

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 occurrences of the string 'logg' ;)

Thanks for submitting a PR, your contribution is really appreciated!

Here's a quick checklist that should be present in PRs:

• Add a new news fragment into the changelog folder
  • name it $issue_id.$type for example (588.bugfix)
  • if you don't have an issue_id change it to the pr id after creating the pr
  • ensure type is one of removal, feature, bugfix, vendor, doc or trivial
  • Make sure to use full sentences with correct case and punctuation, for example: "Fix issue with non-ascii contents in doctest text files."
• Target: for bugfix, vendor, doc or trivial fixes, target master; for removals or features target features;
• Make sure to include reasonable tests for your change if necessary

Unless your change is a trivial or a documentation fix (e.g., a typo or reword of a small section) please:

• Add yourself to AUTHORS, in alphabetical order;

[coveralls]

Coverage remained the same at 92.588% when pulling 3862b0b on thisch:removelogger into 476d4df on pytest-dev:master.

[coveralls]

Coverage remained the same at 92.588% when pulling 3862b0b on thisch:removelogger into 476d4df on pytest-dev:master.

[nicoddemus]

Thanks @Thisch, it indeed simplifies things! 👍