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

move mmtune into device status so it works with multiple rigs #1516

Merged
merged 2 commits into from
Feb 15, 2016

Conversation

jasoncalabrese
Copy link
Member

No description provided.

@jasoncalabrese jasoncalabrese added this to the 0.9 milestone Feb 15, 2016
@codecov-io
Copy link

Current coverage is 74.83%

Merging #1516 into dev will increase coverage by +0.08% as of 2f10922

@@              dev   #1516   diff @@
=====================================
  Files          94      94       
  Stmts        7929    7929       
  Branches        0       0       
  Methods         0       0       
=====================================
+ Hit          5927    5934     +7
  Partial         0       0       
+ Missed       2002    1995     -7

Review entire Coverage Diff as of 2f10922


Uncovered Suggestions

  1. +0.29% via .../client/boluscalc.js#162...184
  2. +0.26% via .../client/boluscalc.js#303...322
  3. +0.23% via ...lugins/insulinage.js#73...90
  4. See 7 more...

Powered by Codecov. Updated on successful CI builds.

jasoncalabrese added a commit that referenced this pull request Feb 15, 2016
move mmtune into device status so it works with multiple rigs
@jasoncalabrese jasoncalabrese merged commit d9b7cdc into dev Feb 15, 2016
@jasoncalabrese jasoncalabrese deleted the wip/mmtune-per branch February 15, 2016 03:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants