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

Generic link parser captures any text with a period inside #42

Open
jrmontag opened this issue Apr 3, 2019 · 0 comments
Open

Generic link parser captures any text with a period inside #42

jrmontag opened this issue Apr 3, 2019 · 0 comments

Comments

@jrmontag
Copy link
Collaborator

jrmontag commented Apr 3, 2019

Capturing this note for potential future enhancement. The current "generic link" regular expression matches any text of the form foo.bar including missed spaces between sentences, and email domains e.g. gmail.com. It would be helpful to improve the regex to reduce the number of false positive matches. https://github.com/twitterdev/tweet_parser/blob/3435de8367d36b483a6cfd8d46cc28694ee8a42e/tweet_parser/getter_methods/tweet_text.py#L303

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant