-
-
Notifications
You must be signed in to change notification settings - Fork 813
refactor: optimize fuzzy matching with lazy initialization #2523
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
Changes from all commits
1deb1d8
dcc23cf
3ce9cc6
bd881e8
2a88d5c
7ed6637
b74a408
4aac0ec
03a4982
5a1405d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -91,6 +91,17 @@ experiments: | |
| verbose: true | ||
| ``` | ||
|
|
||
| ### `disable-fuzzy` | ||
|
|
||
| - **Type**: `boolean` | ||
| - **Default**: `false` | ||
| - **Description**: Disable fuzzy matching for task names. When enabled, Task will not suggest similar task names when you mistype a task name. | ||
| - **CLI equivalent**: [`--disable-fuzzy`](./cli.md#--disable-fuzzy) | ||
|
|
||
| ```yaml | ||
| disable-fuzzy: true | ||
| ``` | ||
|
|
||
| ### `concurrency` | ||
|
|
||
| - **Type**: `integer` | ||
|
|
@@ -120,6 +131,7 @@ Here's a complete example of a `.taskrc.yml` file with all available options: | |
| ```yaml | ||
| # Global settings | ||
| verbose: true | ||
| disable-fuzzy: false | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I just realized that, we are apparently inconsistency when it comes to
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think the default is to use |
||
| concurrency: 2 | ||
|
|
||
| # Enable experimental features | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should probably wrap this into a mutex.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! I've implemented a Once to be sure this is executed only once