Skip to content

Conversation

@tpapp
Copy link
Owner

@tpapp tpapp commented Dec 12, 2024

Remove legacy ones (still available with an option), test.

Remove legacy ones (still available with an option), test.
@tpapp
Copy link
Owner Author

tpapp commented Dec 12, 2024

@PetrKryslUCSD, this should fix the clickable locations, cf your comment #107 (comment). It would be great if you could test it.

@PetrKryslUCSD
Copy link

PetrKryslUCSD commented Dec 12, 2024

Thanks! I took the file from the PR and load it directly from disk.

I get this.
The first two lines are NOT clickable:
image

@tpapp
Copy link
Owner Author

tpapp commented Dec 13, 2024

@PetrKryslUCSD: I tweaked the regexps to include _, can you please check again?

@PetrKryslUCSD
Copy link

Fantastic, that was it!
image
Many thanks indeed!

@tpapp
Copy link
Owner Author

tpapp commented Dec 13, 2024

Thanks for checking, I really appreciate it. Will merge and release now.

@tpapp tpapp merged commit 3ba8f5d into master Dec 13, 2024
9 checks passed
@tpapp tpapp deleted the tp/fix-error-regexps branch December 13, 2024 16:19
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

Successfully merging this pull request may close these issues.

3 participants