Skip to content
This repository has been archived by the owner on Oct 17, 2018. It is now read-only.

Expose the id of aggregation types #51

Merged
merged 3 commits into from
Jun 13, 2017
Merged

Expose the id of aggregation types #51

merged 3 commits into from
Jun 13, 2017

Conversation

cw9
Copy link
Contributor

@cw9 cw9 commented Jun 12, 2017

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 87.983% when pulling 59739da on chao-aggtype-id into f55a389 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 87.983% when pulling b6b809b on chao-aggtype-id into f55a389 on master.

xichen2020
xichen2020 approved these changes Jun 13, 2017
View reviewed changes
Copy link
Contributor

@xichen2020 xichen2020 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cw9 cw9 merged commit 4503730 into master Jun 13, 2017
@cw9 cw9 deleted the chao-aggtype-id branch June 13, 2017 18:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@xichen2020 xichen2020 xichen2020 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cw9 @coveralls @xichen2020