-
Notifications
You must be signed in to change notification settings - Fork 86
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
Remove print statement from middleware #157
Remove print statement from middleware #157
Conversation
👍 |
Thanks for this one :) |
Anyone who can merge this? Having this print statement is really really annoying. cc: @trallnag @mander1000 |
➕ on this one... have been hunting this down today. |
this was a source of insanity for us! and i'm very happy i found this, but am disappointed that it has not yet been fixed. |
@trallnag @mander1000 Can we merge this please? |
Hello?! @trallnag |
In the meantime, my Splunk is going |
Oh, wow you have True, mine is always False. Lucky you ;) |
That extra letter is going to cost you 😆 |
At least @trallnag moved some files this weekend... hopefully picking this one up soon as well. |
b8dac0b
to
e1669c8
Compare
Codecov Report
@@ Coverage Diff @@
## master #157 +/- ##
==========================================
- Coverage 93.51% 93.48% -0.03%
==========================================
Files 4 4
Lines 262 261 -1
==========================================
- Hits 245 244 -1
Misses 17 17
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
🍾 for everyone! Thanks! |
Thanks for giving a reason to celebrate today 😉 |
Cheers 🍾 |
As mentioned in issue #156 and #151 the middleware prints at sometime.
I simply removed the print statement.