Skip to content
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

BuildEnvironmentWarning logged as error #4978

Closed
humitos opened this issue Dec 10, 2018 · 1 comment · Fixed by #5118
Closed

BuildEnvironmentWarning logged as error #4978

humitos opened this issue Dec 10, 2018 · 1 comment · Fixed by #5118
Labels
Accepted Accepted issue on our roadmap Improvement Minor improvement to code
Milestone
Cleanup

Comments

@humitos
Copy link
Member

humitos commented Dec 10, 2018

After #4804 got merged BuildEnvironmentWarning start being logged again as an error.

This is not a big problem but it's annoying when debugging since it uses our Sentry quota.

Relating to #4590 since that could help us to avoid logging all of these.
@humitos humitos added this to the Cleanup milestone Dec 10, 2018
@dojutsu-user
Copy link
Member

dojutsu-user commented Dec 18, 2018 •
edited
Loading

@humitos
I would like to take this issue up.
Any idea on how to start?

Edit: I'm working on it according to this comment.

@dojutsu-user dojutsu-user mentioned this issue Jan 2, 2019
Merged
@humitos humitos added Improvement Minor improvement to code Accepted Accepted issue on our roadmap labels Jan 9, 2019
@humitos humitos mentioned this issue Jan 16, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Accepted Accepted issue on our roadmap Improvement Minor improvement to code
Projects
None yet
Milestone
Cleanup
Development

Successfully merging a pull request may close this issue.

Logging exceptions rework readthedocs/readthedocs.org
2 participants
@humitos @dojutsu-user