What was the reason for removing a period after the error message?

[gajus]

https://github.com/jscs-dev/jscs-jsdoc/commit/c7684978337ff8679296923e0f971addec376e9e#diff-b42988fd48ef517b52861127d96304b8L80

In this commit you have removed a period after all the error messages. What is the reason for that?

[qfox]

In JSCS for some reason we don't have periods in error messages.

E.g.:

• https://github.com/jscs-dev/node-jscs/blob/master/lib/rules/disallow-arrow-functions.js#L64
• https://github.com/jscs-dev/node-jscs/search?utf8=%E2%9C%93&q=errors.add

upd:

Guess because of reporting them to user.

validateIndentation: Expected indentation of 6 characters at ./lib/api.js :
    66 |                return cb(node);

Like:

<rule>: <ErrorMessage> at <file> :
  <line> | <code>...

So better to remove this period at the end.

[gajus]

It is somewhat ironic that an error for a rule requireDescriptionCompleteSentence is without a closing period. But if thats the convention, thats the convention.

[qfox]

@gajus up ↑ ;-)