Skip to content

Wip/bewest/separate concerns #33

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
Merged
merged 19 commits into from
Oct 6, 2015
Merged

Wip/bewest/separate concerns #33

merged 19 commits into from
Oct 6, 2015

Conversation

@bewest
Copy link
Member

@bewest bewest commented Sep 27, 2015

Build on @cjo20's work a bit, and cleaning things up a little.

cjo20 and others added 18 commits September 27, 2015 00:29
@cjo20
Adding ns-upload-glucose.sh
0bc352d
@cjo20
In ns-upload-glucose, fix 'type' for sgv entries, set device to show …
482e303 
…it comes from cgm
@cjo20
ns-upload-glucose.sh, note this was written for decocare v0.0.17
b03821e
@bewest
make script names match installed script names
e757efe
@bewest
add more tools to the package/distribution
0826bee
@bewest
separate concerns a bit
dbf8458 
The formatting things are unique, different concern from doing network I/O or
uploading.
@bewest
use stdout by default
65d609f
@bewest
Merge branch 'dev' into wip/bewest/separate-concerns
04da2a0
@bewest
Provide top level exec shell for diyps.
a61cf59 
A little tool to help @scottleibrand save some typing.  Any
tool called `diyps-foo`, eg `diyps-pebble` can be
registered once in openaps as `diyps` device, and then used
many times using `openaps use diyps shell pebble ...` etc.
@bewest
even more helpers
d413a0b
@bewest
allow diyps wrapper to handle some basic debugging
4816802
@bewest
init mm-stick tool
a07e2f9 
To help with common debugging tasks with the usb stick.
@bewest
distribute with package
40a53a5 
Should help @audifile a bit.
@bewest
be strict about errors
f74e54d
@bewest
fix default behavior with no args should not error
839b9d9
@bewest
ifttt tool, rename everything to oref0
c3a821b 
Hat tip @danalewis, @channemann
Also, include @audiefile's tool to post notifications to IFTTT.
It's modified to be usable as `oref0 iftt-notify <endpoint>` as a use for
openaps device, `oref0`.
@bewest
spruce up help output a bit
e81593d
@bewest
Merge branch 'dev' of github.com:openaps/openaps-js into wip/bewest/s…
b38300d 
…eparate-concerns
@bewest
Copy link
Member Author

bewest commented Oct 6, 2015 

$ oref0 
  Usage:
oref0 <cmd>

Valid commands:
  oref0 env - print information about environment.
  oref0 pebble
  oref0 iftt-notify
  oref0 get-profile
  oref0 calculate-iob
  oref0 determine-basal
  oref0 help - this message

@bewest bewest mentioned this pull request Oct 6, 2015
Merged
bewest added a commit that referenced this pull request Oct 6, 2015
@bewest
Merge pull request #33 from openaps/wip/bewest/separate-concerns
1883610 
Wip/bewest/separate concerns Moving to dev for more review.
@bewest bewest merged commit 1883610 into dev Oct 6, 2015
@bewest bewest deleted the wip/bewest/separate-concerns branch October 6, 2015 20:10
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.

3 participants

@bewest @cjo20