Why are sessionSymbols considered suspicious? about codeinspector HOT 2 CLOSED

Quit, Echo, etc. are considered suspicious in package code because they are generally not intended to be in package code, and their presence is a bug.

This can happen because they were left in during debugging

This can also happen because code is copied from a notebook, where it is more acceptable to use Quit, Echo, etc.

The presence of these symbols is not always a problem, which is why this rule has confidence of 0.55.

Right now, the rule is only looking at the presence of the symbol itself.

When more specific patterns are developed and the context of the symbols in the surrounding code is better understood, then there will be fewer false positives and I will increase the confidence in the rule.

from codeinspector.

This is useful. I've left a stray Echo in there before.

from codeinspector.