Skip to content

Add Groups Implementation to Syncano Core#309

Merged
jonny22094 merged 5 commits into
masterfrom
feat/core-lib-groups
Oct 30, 2018
Merged

Add Groups Implementation to Syncano Core#309
jonny22094 merged 5 commits into
masterfrom
feat/core-lib-groups

Conversation

@jonny22094

Copy link
Copy Markdown
Contributor

No description provided.

@jonny22094 jonny22094 self-assigned this Oct 29, 2018
@jonny22094 jonny22094 requested a review from mkucharz October 29, 2018 17:33
@codecov-io

codecov-io commented Oct 29, 2018

Copy link
Copy Markdown

Codecov Report

Merging #309 into master will decrease coverage by 0.92%.
The diff coverage is 84.48%.

@@            Coverage Diff             @@
##           master     #309      +/-   ##
==========================================
- Coverage   39.42%   38.49%   -0.93%     
==========================================
  Files          87       99      +12     
  Lines        2765     4159    +1394     
  Branches       10       10              
==========================================
+ Hits         1090     1601     +511     
- Misses       1674     2557     +883     
  Partials        1        1

Comment thread packages/lib-js-core/src/groups.js Outdated
Comment thread packages/lib-js-core/src/groups.js Outdated
@mkucharz

Copy link
Copy Markdown
Member

Other than those minor things, looks good. Way to go @jonny22094 ;)

@jonny22094 jonny22094 merged commit 0623427 into master Oct 30, 2018
@jonny22094 jonny22094 deleted the feat/core-lib-groups branch October 30, 2018 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants