Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Static predicate field names are inconsistent #1624

Open
haykam821 opened this issue Jul 28, 2020 · 8 comments
Open

Static predicate field names are inconsistent #1624

haykam821 opened this issue Jul 28, 2020 · 8 comments
Labels
discussion vote A vote on a name refactor

Comments

@haykam821
Copy link
Contributor

haykam821 commented Jul 28, 2020

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

@haykam821 haykam821 added discussion vote A vote on a name refactor labels Jul 28, 2020
@liach
Copy link
Contributor

liach commented Jul 28, 2020

I prefer only FILTER/CHECKER suffix without IS prefix.

@YanisBft
Copy link
Contributor

I would prefer PREDICATE suffix without IS prefix

@Earthcomputer
Copy link
Contributor

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.

@TheBrokenRail
Copy link
Contributor

I also prefer IS_FOOD or IS_FOOD_PREDICATE

@Runemoro
Copy link
Contributor

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

@liach
Copy link
Contributor

liach commented Aug 19, 2020

Yes, I also prefer the IS_ suffix.

concern

@Sollace
Copy link
Contributor

Sollace commented Apr 9, 2021

'Is' or 'can' prefix only: IS_FOOD

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

@liach
Copy link
Contributor

liach commented Apr 9, 2021

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion vote A vote on a name refactor
Projects
None yet
Development

No branches or pull requests