yarn icon indicating copy to clipboard operation
yarn copied to clipboard

Static predicate field names are inconsistent

Open haykam821 opened this issue 3 years ago • 8 comments

There isn't a consistent name scheme for static predicate fields. We should decide on one of these formats:

haykam821 avatar Jul 28 '20 08:07 haykam821

I prefer only FILTER/CHECKER suffix without IS prefix.

liach avatar Jul 28 '20 08:07 liach

I would prefer PREDICATE suffix without IS prefix

YanisBft avatar Jul 28 '20 23:07 YanisBft

Imo a predicate is a function, so IS_FOOD makes the most sense to me, and fits with the style I tend to go with in my personal code.

Earthcomputer avatar Aug 17 '20 21:08 Earthcomputer

I also prefer IS_FOOD or IS_FOOD_PREDICATE

TheBrokenRail avatar Aug 18 '20 19:08 TheBrokenRail

Yes, I also prefer the IS_ suffix. More generally I think we should name Predicates and Functions as if they were methods.

Runemoro avatar Aug 19 '20 05:08 Runemoro

Yes, I also prefer the IS_ suffix.

concern

liach avatar Aug 19 '20 06:08 liach

'Is' or 'can' prefix only: IS_FOOD

I vote for IS_ prefix only. It's just the most concise.

Sollace avatar Apr 09 '21 17:04 Sollace

looking back, now i support is prefix. or we can alternatively have a 3rd person present tense verb, like HAS_EFFECT, REQUIRES_SOME_PERMISSION, etc.

liach avatar Apr 09 '21 23:04 liach