-
Notifications
You must be signed in to change notification settings - Fork 28
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
disable "nobody's around" message #108 #110
Conversation
1 similar comment
Thanks, I agree -- i think it's causing disruption currently, and until we can resolve with something like #109 we need to put a hold on this behavior. Would this result in plotsbot sending an empty string message somehow? Could we instead comment out this line? Line 18 in d87ef5d
and then comment out the corresponding test entirely for the time being? Some test frameworks allow for things like |
@jywarren sorry, I should have mentioned that I tested it. No, an empty string isn't sent. Nothing is sent. Silence is golden. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM until we figure out a better solution.
@icarito could we deploy this, or does it deploy automatically now? I see motion at https://jenkins.laboratoriopublico.org/ ... cool! |
Yup it's automatic. :-)
…On 01/05/18 17:56, Jeffrey Warren wrote:
@icarito <https://github.com/icarito> could we deploy this, or does it
deploy automatically now? I see motion at
https://jenkins.laboratoriopublico.org/ ... cool!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#110 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAMMS9f8gkV9PKmjSbTWFly70hgH0k28ks5tuOgogaJpZM4Ttbhq>.
|
Nice! Thanks, all! |
I've described in #108 how plotbot always insists on getting the last word. In this pull request I'm attempting to disable the "nobody's around" message. We can rethink this feature in the future.