Update to nom 7 - Githubissues

zslayton / cron

A cron expression parser in Rust

Apache License 2.0

376 stars 69 forks source link

Update to nom 7 #90

Closed nickelc closed 3 years ago

nickelc commented 3 years ago

The PR updates nom to v7. Since v7 removed its parser macros I replaced them with functions.

As a side note, the initial update to nom v5 breaks like 35+ tests for some reason but I didn't look into it after realizing that replacing the macro usage fixed everything.

zslayton commented 3 years ago

This is excellent, thanks for the PR!