Skip to content
This repository has been archived by the owner on Sep 14, 2023. It is now read-only.

update strsim to 0.8 #246

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ignatenkobrain
Copy link
Contributor

No description provided.

@BurntSushi
Copy link
Member

@ignatenkobrain Hmm, is this worth doing? It looks like strsim 0.8 requires a newer version of Rust, so this probably means a docopt 1.1 release. I guess I'm fine with that, but I also feel like strsim 0.7 is doing fine?

@ignatenkobrain
Copy link
Contributor Author

As usual, I don't want to package multiple versions of package in Fedora and I would prefer to not carry downstream patch.

@BurntSushi
Copy link
Member

I don't know whether that's a sustainable policy moving forward long term, but I'm happy to keep pushing. I filed an issue with strsim for now: rapidfuzz/strsim-rs#26 --- Once they add a Rust version to their CI config, can you add that same version to this PR? After that, I'll do a 1.1 release.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants