Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fix notif kws that start/end with non-word chars #2500

Merged
merged 3 commits into from Oct 5, 2017

Conversation

Projects
None yet
2 participants
Member

dbkr commented Oct 5, 2017 edited by ara4n

Only prepend / append word boundary characters if the search
expression starts or ends with a word character, otherwise they
don't work because there's no word boundary between whitespace and
a non-word char.

dbkr added some commits Oct 5, 2017

Fix notif kws that start/end with non-word chars
Only prepend / append word bounary characters if the search
expression starts or ends with a word character, otherwise they
don't work because there's no word bounary between whitespace and
a non-word char.

@ara4n ara4n self-assigned this Oct 5, 2017

Owner

ara4n commented Oct 5, 2017

lgtm!

@ara4n ara4n assigned dbkr and unassigned ara4n Oct 5, 2017

@dbkr dbkr merged commit 44f8e38 into develop Oct 5, 2017

6 of 8 checks passed

Sytest Dendron (Commit) Build #2706 origin/dbkr/fix_word_boundary_mentions failed in 3 min 21 sec
Details
Sytest Dendron (Merged PR) Build finished.
Details
Sytest Postgres (Commit) Build #3539 origin/dbkr/fix_word_boundary_mentions succeeded in 3 min 31 sec
Details
Sytest Postgres (Merged PR) Build finished.
Details
Sytest SQLite (Commit) Build #3634 origin/dbkr/fix_word_boundary_mentions succeeded in 1 min 47 sec
Details
Sytest SQLite (Merged PR) Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@dbkr dbkr deleted the dbkr/fix_word_boundary_mentions branch Oct 17, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment