un-deprecation using npm deprecate from doc example causes anxiety

[ungoldman]

Hello!

Doc at https://docs.npmjs.com/cli/deprecate says:

To un-deprecate a package, specify an empty string ("") for the message argument.

Originally I was going to say it's not removing deprecation, just removing the deprecation message, e.g.

npm WARN deprecated derp@0.1.8: deprecation message

becomes

npm WARN deprecated derp@0.1.8:

However, copying it from the doc page I now realize there shouldn't be a space in the empty string (obviously I guess), but reading it on the doc page I had not gotten that impression:

This is on a mac running latest chrome. I guess it's just a weird font rendering issue but it's enough to have fooled me into running npm deprecate derp@0.1.8 " " to try to remove it.

Sorry if this isn't real issue material but it's an understandable misreading, right?

Side-note: would it make sense to treat " " as an empty string and remove deprecation? It's not falsy but it sure is empty.

[othiym23]

I think " " should probably be a validation error, just because it's mildly magical to treat it as the same as "". How would you suggest we make the documentation clearer?

[ungoldman]

@othiym23 wrapping it in a code block ("") might be enough to clarify it for dummies like me.

[ungoldman]

Oh weird, it already is wrapped in a code block.

<p>To un-deprecate a package, specify an empty string (<code>""</code>) for the <code>message</code> argument.</p>

I guess it's just the font that's a bit easy to misread.

[othiym23]

I think I'm going to call this close enough to resolved to be done. If you come up with a better way to show that empty string, I'd be happy to land a patch.