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.

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

Log exception content as Important rather than error #1288

Merged
merged 1 commit into from Dec 26, 2017

Conversation

2 participants
@peppy
Member

peppy commented Dec 25, 2017

This allows discerning between description and raw exception outputs when displaying to the end user.

Does not affect underlying logging to file.

Log exception content as Important rather than error
This allows discerning between description and raw exception outputs when displaying to the end user.

Does not affect underlying logging to file.

@peppy peppy referenced this pull request Dec 25, 2017

Merged

Rate limit logged errors displayed as notifications #1745

2 of 2 tasks complete

@smoogipoo smoogipoo merged commit 46d4704 into ppy:master Dec 26, 2017

1 check passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details

@peppy peppy deleted the peppy:exception-logging branch Jan 11, 2018

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