Skip to content

Update the URL regexes matching urls starting with a vulgar www.#1185

Merged
lfoppiano merged 3 commits intomasterfrom
update-regexes-urls
Oct 23, 2024
Merged

Update the URL regexes matching urls starting with a vulgar www.#1185
lfoppiano merged 3 commits intomasterfrom
update-regexes-urls

Conversation

@lfoppiano
Copy link
Copy Markdown
Member

@lfoppiano lfoppiano commented Oct 20, 2024

We added a new regex that provide an alternative matching for the URLs (I did not dear to modify the body of the existing one)

@coveralls
Copy link
Copy Markdown

coveralls commented Oct 20, 2024

Coverage Status

coverage: 40.755% (+0.003%) from 40.752%
when pulling 40a1742 on update-regexes-urls
into d2f0cdc on master.

@kermitt2
Copy link
Copy Markdown
Collaborator

It looks good to me !

@lfoppiano lfoppiano marked this pull request as ready for review October 21, 2024 15:40
@lfoppiano lfoppiano added this to the 0.8.2 milestone Oct 21, 2024
@lfoppiano lfoppiano merged commit be44579 into master Oct 23, 2024
@lfoppiano lfoppiano deleted the update-regexes-urls branch October 23, 2024 08:07
@lfoppiano lfoppiano linked an issue Oct 31, 2024 that may be closed by this pull request
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.

some URLs are not extracted in DAS

3 participants