Skip to content

Comments

Let cron depend on interface for parsers instead of fixed parser#243

Merged
robfig merged 1 commit intorobfig:masterfrom
yvanoers:master
Nov 25, 2019
Merged

Let cron depend on interface for parsers instead of fixed parser#243
robfig merged 1 commit intorobfig:masterfrom
yvanoers:master

Conversation

@yvanoers
Copy link
Contributor

@yvanoers yvanoers commented Oct 9, 2019

This change allows cron to use another or extended parser instead of the one that comes in the package.

@yvanoers
Copy link
Contributor Author

yvanoers commented Oct 9, 2019

@robfig I'm hoping you will consider this PR. If anything needs to be changed for this to get accepted, please let me know.

@vcastellm
Copy link

It is a non invasive change that will allow us and others to implement our own custom parsers. Thanks @robfig

@robfig robfig merged commit b7cc47d into robfig:master Nov 25, 2019
@robfig
Copy link
Owner

robfig commented Nov 25, 2019

Seems reasonable. Thanks!

haiheipijuan pushed a commit to haiheipijuan/cron that referenced this pull request Jun 15, 2021
Let cron depend on interface for parsers instead of fixed parser
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants