Skip to content

Commit 9c74ec1

Browse files
authored
Merge pull request #1334 from pjkaufman/master
Fix Non-Latin Characters Not Working for `remove-consecutive-list-markers`
2 parents 34d5c2c + 4528a46 commit 9c74ec1

File tree

2 files changed

+23
-1
lines changed

2 files changed

+23
-1
lines changed
Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
import RemoveConsecutiveListMarkers from '../src/rules/remove-consecutive-list-markers';
2+
import dedent from 'ts-dedent';
3+
import {ruleTest} from './common';
4+
5+
ruleTest({
6+
RuleBuilderClass: RemoveConsecutiveListMarkers,
7+
testCases: [
8+
{ // accounts for https://github.com/platers/obsidian-linter/issues/1278
9+
testName: 'Make sure consecutive list markers are removed when dealing with non-latin characters',
10+
before: dedent`
11+
- - test content
12+
- - тест content
13+
- - 测试 content
14+
`,
15+
after: dedent`
16+
- test content
17+
- тест content
18+
- 测试 content
19+
`,
20+
},
21+
],
22+
});

src/rules/remove-consecutive-list-markers.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ export default class RemoveConsecutiveListMarkers extends RuleBuilder<RemoveCons
1919
return RemoveConsecutiveListMarkersOptions;
2020
}
2121
apply(text: string, options: RemoveConsecutiveListMarkersOptions): string {
22-
return text.replace(/^([ |\t]*)- - \b/gm, '$1- ');
22+
return text.replace(/^([ |\t]*)- - (\p{L})/gmu, '$1- $2');
2323
}
2424
get exampleBuilders(): ExampleBuilder<RemoveConsecutiveListMarkersOptions>[] {
2525
return [

0 commit comments

Comments
 (0)