New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Removed double hooking of CommandService log event #1225

Merged

foxbot merged 2 commits into discord-net:dev from k-boyle:patch-1

Dec 31, 2018

Contributor

k-boyle commented Dec 31, 2018

The log event is already hooked in Line 38 of program.cs


          Removed double hooking of CommandService log event

798ff50

The log event is already hooked in Line 38 of program.cs

Member

foxbot commented Dec 31, 2018

should also remove LogAsync from CommandHandlingService if it will be unused


          Removed redundant log method

63d9812

foxbot approved these changes

View reviewed changes

foxbot merged commit 552f34c into discord-net:dev

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

Labels

None yet