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
Search for [^A-Za-z0-9] instead of the unsupported \W #350
Conversation
I'm not sure what code I wrote actually. I hope a Ruby developer understand my intention. lol. |
app/controllers/github_controller.rb
Outdated
@@ -85,6 +85,7 @@ def pull_request_hook | |||
|
|||
keywords.each do |keyword| | |||
keyword = keyword[0] | |||
keyword.gsub! '\W', '[ -]' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does [^A-Za-z0-9]
work? If so, we should use that since it’s more correct.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add an underscore to that class, but yeah
@ArtOfCode- @j-f1 Yep. Changes made. |
Note: I do intend to review this, but a scroll through the changes tab doesn't give me confidence it's going to work... search is complicated and I'd like to actually pull and test locally before I give this a pass. |
Does the |
@j-f1 I'm afraid it's because MySQL does not support |
MS has commented a lot of "0 TPs, 0 FPs", in Smokey's PRs. This should fix that wrong search.
P.S. This is my absolutely first lines of Ruby code so there may be very stupid syntax or usage errors. Please check them, Thanks.