Skip to content

Add logger parameter for the LoggingEventHandler.#676

Merged
BoboTiG merged 3 commits intogorakhargosh:masterfrom
Sraw:LoggingEventHandler
Jul 12, 2020
Merged

Add logger parameter for the LoggingEventHandler.#676
BoboTiG merged 3 commits intogorakhargosh:masterfrom
Sraw:LoggingEventHandler

Conversation

@Sraw
Copy link
Copy Markdown
Contributor

@Sraw Sraw commented Jul 11, 2020

As the title says, this PR adds a new parameter to the LoggingEventHandler so that it won't use root logger anymore if a logger is passed in.

@BoboTiG
Copy link
Copy Markdown
Collaborator

BoboTiG commented Jul 11, 2020

Thanks for the PR @Sraw.

Again, if you could add a line in the changelog, it would be great :)

@Sraw
Copy link
Copy Markdown
Contributor Author

Sraw commented Jul 11, 2020

Hi, I've updated the changelog for both PRs.

@BoboTiG
Copy link
Copy Markdown
Collaborator

BoboTiG commented Jul 11, 2020

Well, then there is a conflict to fix :)

@Sraw
Copy link
Copy Markdown
Contributor Author

Sraw commented Jul 11, 2020

Done!

@BoboTiG BoboTiG merged commit e985b67 into gorakhargosh:master Jul 12, 2020
@BoboTiG
Copy link
Copy Markdown
Collaborator

BoboTiG commented Jul 12, 2020

Thanks :)

CCP-Aporia pushed a commit to CCP-Aporia/watchdog that referenced this pull request Aug 13, 2020
* Add logger parameter for the LoggingEventHandler.

* update changelog.rst
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

2 participants