Closed shouvSarker closed 3 years ago
Just putting it up to fish opinions about the change. Should resolve #2 when merged. I will be adding comments, updating readme and modify the shell script in the meantime.
LGTM
Looks like the conflicts with main
might take a while to resolve, though. I guess the first step would be to run Prettier on forecast.ts
from main
. Hopefully it won't be so bad after that.
LGTM
Looks like the conflicts with
main
might take a while to resolve, though. I guess the first step would be to run Prettier onforecast.ts
frommain
. Hopefully it won't be so bad after that.
@kn-ad merge into main completed! Feel free to do a sanity check, I will update readme and then should be all good to merge.
Forecasting now supports custom time intervals with either weeks or days. Formatting has also been updated.