-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
cargo: restrict unicode-segmentation to ~1.1.0 #967
cargo: restrict unicode-segmentation to ~1.1.0 #967
Conversation
@mgeisler, thanks for your PR! By analyzing the history of the files in this pull request, we identified @jimmycuadra, @kbknapp and @tormol to be potential reviewers. |
The newly released version 1.2.0 of unicode-segmentation adds code that use the "?" operator, which in turn requires Rust 1.13.0. However, clap currently still works with Rust 1.11.0 and this caused build failures: https://travis-ci.org/kbknapp/clap-rs/jobs/235010822 The changes since 1.1.0 seem to be related cursors/iterators and I think clap can work fine without them.
7ad588d
to
b05a3eb
Compare
1 similar comment
Is there a way to get this mention bot thing to stop mentioning me? I tried blocking its account but I still got notified here. |
@jimmycuadra I apologise I'll set it up to only ping me or collaborators |
Hi @kbknapp, any change to get this merged? |
Apologies, I'm out of town right now so this slipped my mind. @homu r+ |
📌 Commit b05a3eb has been approved by |
…r=kbknapp cargo: restrict unicode-segmentation to ~1.1.0 Version 1.2.0 of unicode-segmentation adds code that use the "?" operator, which in turn requires Rust 1.13.0. However, clap currently still works with Rust 1.11.0 and this caused build failures: https://travis-ci.org/kbknapp/clap-rs/jobs/235010822 The changes since 1.1.0 seem to be related cursors/iterators and I think clap can work fine without them. This was found as part of #845. See also unicode-rs/unicode-segmentation#26. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/kbknapp/clap-rs/967) <!-- Reviewable:end -->
💥 Test timed out |
Thanks for the approval! :-) |
Version 1.2.0 of unicode-segmentation adds code that use the "?"
operator, which in turn requires Rust 1.13.0. However, clap currently
still works with Rust 1.11.0 and this caused build failures:
https://travis-ci.org/kbknapp/clap-rs/jobs/235010822
The changes since 1.1.0 seem to be related cursors/iterators and I
think clap can work fine without them.
This was found as part of #845. See also unicode-rs/unicode-segmentation#26.
This change is