-
Notifications
You must be signed in to change notification settings - Fork 216
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
Is println bad? #161
Comments
It is just an example. There is nothing wrong with println. However, for production code, there are better solutions, for example slf4j or log4j. So, a number of project that I have been involved with discourage the use of println. See, for example http://stackoverflow.com/questions/2750346/logger-vs-system-out-println |
@matthewfarwell Thanks. Good advice. |
There is
Probably it should be in the scalastyle default configuration: the warning appears only for "bare" |
The default scalastyle config that's provided with it has
println
flagged as a warning in both the token and regex rules:Is this just an example that isn't meant for production use, or is
println
considered bad?The text was updated successfully, but these errors were encountered: