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

Require whitespace before tags in description field #755

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

GeraldJansen
Copy link
Contributor

As of v3.0.3 hamster harvests single word #hash tags from the description field. This PR tweaks the regex used to extract the tags, requiring whitespace before the # character. This will prevent harvesting a tag from a pattern like example.com/page#anchor, as mentioned here: #753 (comment).

This does not affect any of the test cases in test_stuff.py nor the examples from the input.page documentation.

As of v3.0.3 hamster harvests single word #hash tags from the description
field. This PR tweaks the regex used to extract the tags, requiring
whitespace before the # character. This will prevent harvesting a tag from
a pattern like example.com/page#anchor, as mentioned here:
projecthamster#753 (comment).

This does not affect any of the test cases in test_stuff.py nor the example
from the input.page documentation.
Copy link

Automatically generated build artifacts for commit 90faaa7 (note: these links will expire after some time):


To test this PR, download and unzip the flatpak application above and then install with:

flatpak install Hamster.flatpak

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.

1 participant