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

Added aggregate function support #12

Merged
merged 1 commit into from Dec 4, 2014
Merged

Conversation

@programcsharp
Copy link
Contributor

@programcsharp programcsharp commented Dec 3, 2014

Love what you've done here by providing a central repo for this to live! I'm surprised MS hasn't.

Here's a dynamic aggregate function that takes the name of a function to run and the name of the property to run it over and returns the result.

kahanu added a commit that referenced this pull request Dec 4, 2014
Added aggregate function support
@kahanu kahanu merged commit 777a9d4 into zzzprojects:master Dec 4, 2014
@kahanu
Copy link
Collaborator

@kahanu kahanu commented Dec 4, 2014

@programcsharp - I just noticed that the var methods variable isn't being used. Are you aware of this?

@programcsharp
Copy link
Contributor Author

@programcsharp programcsharp commented Dec 4, 2014

I didn't realize that -- it's vestigial from a different approach I was trying. Go ahead and nuke it, or let me know and I can resubmit the PR.

@kahanu
Copy link
Collaborator

@kahanu kahanu commented Dec 4, 2014

Ok, I'll remove it so it doesn't confuse others, and thanks for the contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.