-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
doc: clarify caret requirements #12679
Conversation
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @epage (or someone else) soon. Please see the contribution instructions for more information. Namely, in order to ensure the minimum review times lag, PR authors and assigned reviewers should ensure that the review label (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks way better than I expected! Is this ready to merge?
Thanks. I was expecting some discussion. Go for it if it looks good.
…On Sat, Sep 16, 2023, 4:45 AM Weihang Lo ***@***.***> wrote:
***@***.**** approved this pull request.
This looks way better than I expected! Is this ready to merge?
—
Reply to this email directly, view it on GitHub
<#12679 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA2AEUIWHTZWQGWR5KQCBFTX2VRKPANCNFSM6AAAAAA42OB25U>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Thanks! You've done a great job so I don't have to change any word :) @bors r+ |
☀️ Test successful - checks-actions |
Update cargo 6 commits in d5336f813df39d476e61fc46daabb1446350660a..b4ddf95ad9954118ac0dae835f2966394ad04c02 2023-09-14 19:55:49 +0000 to 2023-09-18 03:48:09 +0000 - doc: differentiate defaults for split-debuginfo (rust-lang/cargo#12680) - feat(cli): Add '-n' to dry-run (rust-lang/cargo#12660) - feat: stabilize credential-process and registry-auth (rust-lang/cargo#12649) - refactor: use `RegistryOrIndex` enum to replace two booleans (rust-lang/cargo#12677) - doc: clarify caret requirements (rust-lang/cargo#12679) - feat(pkgid): Allow incomplete versions when unambigious (rust-lang/cargo#12614) r? ghost
What does this PR try to resolve?
clearing up caret version requirements
possibly fixes #12112
How should we test and review this PR?
expert review...