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

Fix regex for affirm parser #241

Closed
earlduque opened this issue Oct 3, 2023 · 2 comments
Closed

Fix regex for affirm parser #241

earlduque opened this issue Oct 3, 2023 · 2 comments
Assignees

Comments

@earlduque
Copy link
Collaborator

The regex flag on the header is incorrect
!affirm, !affirm @user
should be just !affirm, otherwise it's looking for the exact literal string "!affirm, !affirm @user"

dropping the !affirm @user part seems fine too because the parser doesn't seem to be trying to do anything with the username anyways

@ajcooper72
Copy link
Contributor

I'll fix this up now.

@SapphicFire
Copy link
Contributor

Closing as @ajcooper72 has addressed

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

No branches or pull requests

3 participants