Skip to content

Conversation

@vkatsuba
Copy link
Collaborator

  • Added config option default_dictionary for use any custom dictionary
  • Added config option additional_dictionaries for expand exist dictionary

If set default_dictionary only - will be used default_dictionary instead of the dictionary that is currently in the source code.
If set additional_dictionaries only - will be used dictionary that is currently in the source code and added additional dictionaries to it as an extension.
If set default_dictionary and additional_dictionaries - will be used default dictionary what was set plus additional dictionaries to it as an extension.

* Added config option 'default_dictionary' for use any custom dictionary
* Added config option 'additional_dictionaries' for expand exist dictionary
Copy link
Member

@elbrujohalcon elbrujohalcon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A minimal spec change and some refactoring, but the PR in general looks good.

Copy link
Member

@elbrujohalcon elbrujohalcon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent!

@elbrujohalcon elbrujohalcon merged commit d135010 into main Nov 15, 2021
@elbrujohalcon elbrujohalcon deleted the additional_dict_params branch November 15, 2021 11:38
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.

3 participants