Replace any possible escape sequences in external text inserted into …
…messages
(This is a PR against another PR, https://github.com/travis-ci/casher/pull/49)
I'm sorry, but I don't understand the rationale for this. We control what arguments are sent to these functions, and we should be reasonably certain that the strings don't need escaping.
Some of the insertions contain user data for which we cannot be certain that it doesn't have backslashes. For strings you control, you can be certain, but if you don't escape them, you'll have to make judgements calls on what does and doesn't need escaping. A completely unnecessary maintenance burden and a stealthy source of bugs if this changes in the future.