gorhill / cronexpr

Cron expression parser in Go language (golang)
683 stars 168 forks source link

Return Error Rather than Panic #10

Closed briandowns closed 8 years ago

briandowns commented 8 years ago

Would it be possible to change the cronexpr.MustParse() function to return an error rather than panic? If so, I'll gladly submit a PR. Thanks!

gorhill commented 8 years ago

That's what cronexpr.Parse() does. cronexpr.MustParse() is for when one know the cron expression is valid.

briandowns commented 8 years ago

Damn... I should have read more of the issues filed but more so the docs. :D Thanks for the quick response!