Actually, I plan on removing it altogether.  It just adds confusion and is
simpler to just use moment.  See dexcom-stream for completed version.
reformatISO is unique per parser, wrapping it in another module doesn't
help things, and I discovered potential bugs.  Best to remove it.


On Fri, Jan 31, 2014 at 2:04 PM, Kent Quirk <notifications@github.com>wrote:

> In lib/parse.js:
>
> > @@ -63,7 +70,7 @@ var parsers = {
> >        var data = {
> >          value: bg,
> >          type: 'smbg',
> > -        time: tidepoolDateTime.convertToAdjustedUTC(values[columns['Timestamp']],DATE_TIME_MASK,timezone)
>
> Given that we have an existing datetime library, could we maybe move
> reformatISO into it so that we can concentrate time conversion code there?
> We will definitely be revisiting this and it will be useful not to have to
> chase it all over the place.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/tidepool-org/mmcsv-carelink-data/pull/1/files#r9359276>
> .
>