dates() fails to parse "next week"

dates() fails to parse "next week" properly. It parses it but does not create a date for this.

This library can parse dates well (as does your spacetime library) including next week. https://github.com/wanasit/chrono

Thank you for creating this library. I appreciate your work.

[spencermountain]

thanks Taylor, I’ve always seen date-parsing as the most important thing that compromise should do. I agree with your examples. Right now this library is in ‘quality denial’ stage, but changing this is a priority for me this year, and in the next few months. Help is of course welcome @thegoatherder .

Maybe it’s corny, but i’m particularly excited about having a lot of tests - even though they aren’t passing. Maybe one thing that would be helpful is to add a huge number of exhaustive tests - that’s the only way this thing is going to ever be good.

You mentioned week-startings - we’re using spacetime which supports configuring the start of the week, so that should be easy to do.

ya - the current config options are timezone, and today. But we should support british-endian preference, and week-starting preference immediately.

[spencermountain]

cleaning up old issues, these appear to work now:

console.log(nlp(`next week`).dates().json())
console.log(nlp(`We'll have to catch up some time next week`).dates().json())// - 2020-02-10 to 2020-02-16 (week starts on monday according to ISO8601)
console.log(nlp(`Roadworks will take place in May 2020`).dates().json()) //- 2020-05-01 to 2020-05-31
console.log(nlp(`Coming to cinemas in 2021`).dates().json())  //- 2021-01-01 to 2020-12-31

cheers