Apparently this https://gist.github.com/adamstegman/926858 was to get
RSpec to stfu. But now it calmed down again, so not necessary.
And it affects other things, like swallowing print statements,
causing pry to omit output, and hiding other output that is potentially
legitimate.
However, one nice side effect was that it caught other loudmouths in the dragnet:
ActiveRecord::Migration, apparently there is a globaly flag you set to
tell it to shush.
CodeClimate is logging directly to stderr. You can get it to stfu with:
CodeClimate::TestReporter.configuration.logger = Logger.new('/dev/null')
Put it before the CodeClimate::TestReporter.start I didn't do it b/c
it was doing this before (it runs before output streams were silenced)
so I figured maybe someone wanted it that way on purpose. Plus, you
probably do want it logging in CI, which you'd probably do by setting an
env var there, and then silencing it unless you see the var. Since I
don't have access to your CI, and getting all that config wired up
right tends to be fragile, I figured it'd be best to leave it.
Went with a matrix theme, b/c the vindictive side of me enjoyed the idea
of Thor's stupid mouth getting melted shut.
shaqq
commented
9 years ago
Apparently this https://gist.github.com/adamstegman/926858 was to get RSpec to stfu. But now it calmed down again, so not necessary. And it affects other things, like swallowing print statements, causing pry to omit output, and hiding other output that is potentially legitimate.
However, one nice side effect was that it caught other loudmouths in the dragnet:
shelllike any sensible interface, but it was blowing up pretty hard. Had tried wiping out here https://github.com/erikhuda/thor/blob/53b3fc89ce750258e19c53b1a99b32dd5941451a/lib/thor/shell.rb#L6 but it's passing around classes instead of instances, so its like you have to have layers on layers of fake classes. Tried passing it here https://github.com/erikhuda/thor/blob/53b3fc89ce750258e19c53b1a99b32dd5941451a/lib/thor/shell.rb#L53 but the indirection of who'se inheriting from whom and calling what on what and which vars got set by whatever wherever were just too much for me at this hour, so fuck it. Stub it.CodeClimate::TestReporter.configuration.logger = Logger.new('/dev/null')Put it before theCodeClimate::TestReporter.startI didn't do it b/c it was doing this before (it runs before output streams were silenced) so I figured maybe someone wanted it that way on purpose. Plus, you probably do want it logging in CI, which you'd probably do by setting an env var there, and then silencing it unless you see the var. Since I don't have access to your CI, and getting all that config wired up right tends to be fragile, I figured it'd be best to leave it.Went with a matrix theme, b/c the vindictive side of me enjoyed the idea of Thor's stupid mouth getting melted shut.