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

Allow to opt-out of Chinese/Japanese support #28

Merged
merged 1 commit into from
May 13, 2020

Conversation

Xanewok
Copy link
Contributor

@Xanewok Xanewok commented May 7, 2020

While it's awesome to have, the relevant support doubles the amount of dependencies from 50 to 93, pulling some data/index crates, which can blow up the compilation time considerably, even if the user doesn't require the relevant language support.

It's great to have them enabled by the default but this commit allows to opt out via --no-default-features --features stemmer, without forcing the user to enumerate every language already supported by the rust-stemmers crate.

@Xanewok
Copy link
Contributor Author

Xanewok commented May 7, 2020

Not feeling particularly strong about the stemmers feature, feel free to skip it if you feel like it.

I'd love to land the first commit, which jieba-rs and lindera dependencies being made optional, though ❤️

@Xanewok
Copy link
Contributor Author

Xanewok commented May 11, 2020

Dropped the second commit to only fix the misconfigured features in the manifest.

@mattico mattico merged commit 352131b into mattico:master May 13, 2020
@mattico
Copy link
Owner

mattico commented May 13, 2020

Thank you! I should have noticed that after merging those.

@Xanewok Xanewok deleted the lean-and-mean branch May 13, 2020 16:59
@Xanewok
Copy link
Contributor Author

Xanewok commented May 13, 2020

Thanks for merging and releasing a new version!

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.

2 participants