jamestalmage / supports-hyperlinks

Detect whether a terminal emulator supports hyperlinks
MIT License
54 stars 13 forks source link

fix and tests for 4-digit vte version strings #4

Closed ghost closed 5 years ago

ghost commented 5 years ago

This patch fixes the version string parsing to work with 4-digit $VTE_VERSION strings:

On my system (debian 9) I get:

$ echo $VTE_VERSION
4601

which gets incorrectly reported as supporting hyperlinks.

https://github.com/jamestalmage/supports-hyperlinks/issues/2#issuecomment-450286565

ghost commented 5 years ago

There was an already-failing test (no-color flag disables support) when I checked out the code. That's why the CI is failing.