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

Rename SemImply to SemImplied #233

Open
Tracked by #212
alyst opened this issue Jan 5, 2025 · 1 comment
Open
Tracked by #212

Rename SemImply to SemImplied #233

alyst opened this issue Jan 5, 2025 · 1 comment

Comments

@alyst
Copy link
Contributor

alyst commented Jan 5, 2025

Renaming SemImply to SemImplied (and imply field to implied, imply() call to implied() usw) can potentially resolve some confusion.
imply is a verb, so it implies that the imply() call does some action, or that SemImply are some kind of callable objects, whereas imply() is just the accessor method, and SemImply is not callable.
Also, the docs refer to them as "imply types", and I often find myself stumbling over the grammar of such sentences, because on the first read I treat "imply" as a predicate.
With "implied" there should be less confusion, also SemImplied/implied would have a nice complementarity to SemObserved/observed.

@Maximilian-Stefan-Ernst
Copy link
Collaborator

Thanks, added to the list for v0.3.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants