You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, single-character words are highlighted like ALL_CAPS_NAMES, but in practice they are usually class names instead, such as Q and F objects in Django.
It might be better to change the highlighting rule of constants to require at least two characters instead. (Or even two uppercase characters because names like _N also look like a class. This might just be me though.)
The text was updated successfully, but these errors were encountered:
Yeah, valid point. It really boils down to what is the more common use-case. However, we still plan on trying to make the highlighter more configurable and this may be a good configuration option to add. In the mean time I'll keep this thread open to let anyone else who might have an opinion on the matter chime in.
Currently, single-character words are highlighted like
ALL_CAPS_NAMES
, but in practice they are usually class names instead, such asQ
andF
objects in Django.It might be better to change the highlighting rule of constants to require at least two characters instead. (Or even two uppercase characters because names like
_N
also look like a class. This might just be me though.)The text was updated successfully, but these errors were encountered: