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

Not all URLs are recognized as such. #110

Open
eugene2k opened this issue Jul 16, 2016 · 4 comments
Open

Not all URLs are recognized as such. #110

eugene2k opened this issue Jul 16, 2016 · 4 comments

Comments

@eugene2k
Copy link

Presently lilyterm will recognize a URL such as http://example.com, but not http://example. So URLs such as http://localhost/ or http://author/ (a valid url) aren't recogized.

@bryanparadis
Copy link

The conf file holds the regex for web links. You could change it in there to match your needs. Maybe you could find a good standard regex that also does what you want and suggest it or do a pull request

~/.config/lilyterm/default.conf

@eugene2k
Copy link
Author

Neither my conf file, nor the one in this repository actually hold the url regex.

@bryanparadis
Copy link

I have a large list of regex in my conf file:
username_regex
password_regex
hostname_regex
address_body_regex
address_end_regex
web_match_regex
ftp_match_regex
file_match_regex
email_match_regex

They seem to have been added as a custom regex feature:
6772ca1
e677145

Are you running the current git or an older stable release?

@eugene2k
Copy link
Author

It's an older stable version. I didn't realize the last release was five years ago until now. Thx!

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

2 participants