Skip to content

Conversation

@hoxbro
Copy link
Collaborator

@hoxbro hoxbro commented Jul 2, 2025

The previous regex was too greedy, causing poor extraction when multiple links were provided and only the last one was a .ipynb link.

Previous:

image

New:

image

[HoloViews](https://holoviews.org) ([Interactivity](3_Interactivity.ipynb)).


(\[[^\]]+\]\()([^)\s]+\.ipynb)((?:#[^)\s]*)?\))

(\[.+?\]\()(.+?\.ipynb)(#*?.*?\))

Using: https://regex101.com/

Observed problem:
image

image

@hoxbro hoxbro enabled auto-merge (squash) July 2, 2025 16:22
@hoxbro hoxbro merged commit 03b995e into main Jul 2, 2025
11 checks passed
@hoxbro hoxbro deleted the fix_regex_match branch July 2, 2025 16:24
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.

2 participants