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.

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

Only log to console.error when Raven.debug is true #229

Merged
merged 2 commits into from
Jul 16, 2014

Conversation

zaizhuang
Copy link
Contributor

This allows us to use Raven in production regardless of it being configured or not, without worrying about logging unnecessary errors to the console. Useful when Raven is only configured for a certain percentage of sessions to reduce the number of errors sent to sentry.

If debugging is required, just set Raven.debug = true.

@mattrobenolt
Copy link
Contributor

To maintain existing behavior, I'd prefer then to have Raven.debug default to true, and we can be explicit to disable it.

Then we can change this behavior with a larger version bump and mention it in the changelog.

How does that sound?

@zaizhuang
Copy link
Contributor Author

Sure works for me

@mattrobenolt
Copy link
Contributor

Thanks! 🍰

mattrobenolt added a commit that referenced this pull request Jul 16, 2014
Only log to console.error when Raven.debug is true
@mattrobenolt mattrobenolt merged commit fcad13a into getsentry:master Jul 16, 2014
kamilogorek pushed a commit that referenced this pull request Jun 12, 2018
* Version 1.0.0-beta.0

* Make version validity regex understand beta version naming scheme
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants