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

V0.9 staging #67

Closed
wants to merge 9 commits into from
Closed

V0.9 staging #67

wants to merge 9 commits into from

Conversation

TedDriggs
Copy link
Owner

Prepare for v0.9

This will make internal refactoring easier going forward.

This change also removes dead code exposed by making the crates private.

Fixes #58
* Replace trim_left_matches with trim_start_matches
* Replace trim_right_matches with trim_end_matches
* Remove unnecessary lifetimes
* Allow len without is_empty, as Error can never be empty

Relates to #46
@TedDriggs TedDriggs added this to the 0.9.0 milestone Mar 19, 2019
@TedDriggs TedDriggs self-assigned this Mar 19, 2019
@TedDriggs
Copy link
Owner Author

I'm wondering if I should defer enabling suggestions by default so that 0.9 can avoid the jump to 1.33. That seems like it'd be an issue for a number of crates, and if there's an older version of strsim available then it's not necessary.

@TedDriggs
Copy link
Owner Author

Dropping strsim to 0.7.0 works on 1.18, so I'm closing this PR and opening a new one.

@TedDriggs TedDriggs closed this Mar 20, 2019
@TedDriggs TedDriggs deleted the v0.9-staging branch March 20, 2019 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant