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.

Sign up for GitHub

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

Jump to bottom

Develop #12

Merged
merged 9 commits into from
Jan 17, 2018
Merged

Develop #12

merged 9 commits into from
Jan 17, 2018

Conversation

piyushgupta1
Copy link
Owner

No description provided.

Piyush Gupta and others added 4 commits January 15, 2018 15:01
chore(tsconfig): More checjs
e478aad 
better checks in TSCONFIG
feat($core): Class based to function Based
48ad80f 
Rewrite the code base to move away from class based inplemention to funtion based one

BREAKING CHANGE: API change
Merge branch 'feature/f-p' into develop
d5cc011
Merge branch 'master' into develop
eb010ff
@coveralls
Copy link

Coverage Status

Coverage decreased (-7.1%) to 86.574% when pulling eb010ff on develop into b729e8e on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-7.1%) to 86.574% when pulling eb010ff on develop into b729e8e on master.

Piyush Gupta added 4 commits January 17, 2018 11:51
test($core): Added Test Cases
913a07a
fix($group): Fix Token Name for Nesting
af93429 
As Nesting is used. Aggregation function token were mismatched. Changed to 'nestingApplied'
fix($core): Error Handling
03eb5e3 
When error was thrown. Subsequent data worker invocations were inconsistent
fix($aggregate): fix lang and coverage
7fb1afd 
Removed redundant ifs, And Changed language and error type of some errors for consistency sake
@coveralls
Copy link

Coverage Status

Coverage increased (+2.1%) to 95.833% when pulling 7fb1afd on develop into b729e8e on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+2.1%) to 95.833% when pulling 26ce44d on develop into b729e8e on master.

@piyushgupta1 piyushgupta1 merged commit 28295f7 into master Jan 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@piyushgupta1 @coveralls