Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incorrectly parses shorthand with suffix e.g. "@dailyBla" #86

Closed
geovie opened this issue Apr 6, 2021 · 1 comment · Fixed by #91
Closed

Incorrectly parses shorthand with suffix e.g. "@dailyBla" #86

geovie opened this issue Apr 6, 2021 · 1 comment · Fixed by #91

Comments

@geovie
Copy link

geovie commented Apr 6, 2021

Hi, first thanks for this crate. When testing it I've found that a shorthand (e.g. @weekly) followed by any characters (e.g. @weeklyFoo) will be parsed as @weekly.

To be more strict it probably should return an error in this case or what do you think?

@zslayton
Copy link
Owner

zslayton commented Apr 7, 2021

Thanks for reporting this. I agree that superfluous trailing characters should be rejected with an error.

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 a pull request may close this issue.

2 participants