Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
    Michael Demmer
    @demmer
    not sure what you mean
    Michael Demmer
    @demmer
    one issue with the above plan is that :now: - :forever: != :beginning:
    Michael Demmer
    @demmer
    This message was deleted
    Michael Demmer
    @demmer
    moreover what are the semantics of :beginning: - :1s: ?
    Will Welch
    @welch
    that one is easier, both moment and javascript reject msecs outside the +-1Mdays range as being invalid.
    without infinity semantics, we need to forbid math with forever and only use it for comparisons
    Will Welch
    @welch
    an easy way to handle all of these is to check if the constructed moment is valid -- both your examples produce invalid moments without any extra work on our part
    Michael Demmer
    @demmer
    i suppose so
    another thought — since the semantics of beginning and end are less than obvious, what if we allow them to be specified by the caller in options to the code generator?
    Will Welch
    @welch
    neutral to negative, as i don't see how to make infinity semantics work (if that's the alternate you're wanting to support).
    Michael Demmer
    @demmer
    juttle/moment-parser@cb26693
    With this change i can remove the copy of moment-parser from juttle and still have it use the infinite moments like it does now. We can then separately consider whether the semantics for beginning and end should change in juttle.
    Will Welch
    @welch
    not a fan, but it's reasonable to supply optional values for the literals, as with 'now'
    Michael Demmer
    @demmer
    :)
    come to think of it though — we should be consistent and have both options be moment objects, not a moment for now but an integer values for beginning / end
    i’ll change that now
    Michael Demmer
    @demmer
    done. see juttle/moment-parser#27
    Will Welch
    @welch
    yep
    Michael Demmer
    @demmer
    excellent. i’ll merge this and push a 0.3.0 release
    Will Welch
    @welch
    to infinity and beyond!
    there's a quick pr languishing
    Michael Demmer
    @demmer
    not any more
    Will Welch
    @welch
    whoosh
    Michael Demmer
    @demmer
    gotta run but will do this shortly
    Michael Demmer
    @demmer
    @welch juttle/juttle#88 contains the juttle changes to extirpate moment-parser from juttle and use the external library instead. Take a look if you’re interested.
    Michael Demmer
    @demmer
    @welch FYI as part of the discussion on removing “last 3 months” from the grammar I noticed there is a subtle difference that I didn’t realize. It seems like we no longer support expressions like “day 1 of 3 months ago”, which used to be the equivalent moment and seems like something we should still support.
    Will Welch
    @welch
    that's unintentional, i'll take a look next time i come up for air
    Michael Demmer
    @demmer
    Cool