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

Add missing fragment specifier to a clap_app! rule. #857

Merged
merged 1 commit into from
Feb 18, 2017

Conversation

eddyb
Copy link
Contributor

@eddyb eddyb commented Feb 18, 2017

Introduced in #238, this bug affects both clap 1.* and 2.* and was found by rust-lang/rust#39419.
I'd suggest releasing not only 2.20.5, but also 1.5.6, to cover downstream crates still on 1.5.5.

@eddyb
Copy link
Contributor Author

eddyb commented Feb 18, 2017

cc @james-darkfox @jseyfried

@eddyb
Copy link
Contributor Author

eddyb commented Feb 18, 2017

Oh and if anyone knows how to trigger this path to test it, that'd be helpful.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 90.659% when pulling f28b235 on eddyb:patch-1 into cb81ce5 on kbknapp:master.

@kbknapp
Copy link
Member

kbknapp commented Feb 18, 2017

Thanks for catching this and letting me know! I'll push out the two new versions.

@homu r+

@homu
Copy link
Contributor

homu commented Feb 18, 2017

📌 Commit f28b235 has been approved by kbknapp

homu added a commit that referenced this pull request Feb 18, 2017
Add missing fragment specifier to a clap_app! rule.

Introduced in #238, this bug affects both `clap` `1.*` and `2.*` and was found by rust-lang/rust#39419.
I'd suggest releasing not only `2.20.5`, but also `1.5.6`, to cover downstream crates still on `1.5.5`.
@homu homu merged commit f28b235 into clap-rs:master Feb 18, 2017
@homu
Copy link
Contributor

homu commented Feb 18, 2017

⚡ Test exempted - status

@eddyb eddyb deleted the patch-1 branch February 18, 2017 16:41
@kbknapp
Copy link
Member

kbknapp commented Feb 18, 2017

v2.20.5 is out and v1.5.6 is in PR right now

wildwestrom added a commit to wildwestrom/clap that referenced this pull request Jul 29, 2022
Because why the hell not?

1.6.0 (2022-07-05)
New features:
    FEATURE clap-rs#832: Clarify that Captures::len includes all groups, not just matching groups.
    FEATURE clap-rs#857: Add an ExactSizeIterator impl for SubCaptureMatches.
    FEATURE clap-rs#861: Improve RegexSet documentation examples.
    FEATURE clap-rs#877: Upgrade to Unicode 14.
Bug fixes:
    BUG clap-rs#792: Fix error message rendering bug.

1.5.6 (2022-05-20)
    BUG clap-rs#680: Fixes a bug where [[:alnum:][:^ascii:]] dropped [:alnum:] from the class.
    BUG clap-rs#859: Fixes a bug where Hir::is_match_empty returned false for \b.
    BUG clap-rs#862: Fixes a bug where 'ab??' matches 'ab' instead of 'a' in 'ab'.
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.

4 participants