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

[FEAT] Rename DD.Aggregation NotEmpty operator. #523

Closed
HavenDV opened this issue Oct 23, 2021 · 1 comment · Fixed by #583
Closed

[FEAT] Rename DD.Aggregation NotEmpty operator. #523

HavenDV opened this issue Oct 23, 2021 · 1 comment · Fixed by #583

Comments

@HavenDV
Copy link
Contributor

HavenDV commented Oct 23, 2021

I see NotEmpty DD.Aggregation operator, but unfortunately this is not usable due to the intersection of namespaces. I think you should come up with a new name for this operator.
image

The simplest thing is to rename it to IsNotEmpty. This is consistent with IsEmpty and is not busy.

glennawatson pushed a commit that referenced this issue May 5, 2022
* Rename DD.Aggregation NotEmpty operator. Fixes  #523

* Binding list insert issue. Fixes #507

* Add assembly name to ExpressionBuilder cache key. Fixes #434

* Make OnBeingRemoved behave the same for cache and list.  Fixes #268
@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant