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

Introduce -Zprecise-pre-release unstable flag #13296

Merged
merged 1 commit into from
Jan 15, 2024

Conversation

eopb
Copy link
Contributor

@eopb eopb commented Jan 14, 2024

Tracking Issue: #13290

This change introduces the feature but does not yet attempt an implementation. The actual implementation will happen in future PRs.

r? @epage

@rustbot rustbot added A-documenting-cargo-itself Area: Cargo's documentation A-unstable Area: nightly unstable support S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 14, 2024
Copy link
Member

@weihanglo weihanglo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the effort!

tests/testsuite/precise_pre_release.rs Outdated Show resolved Hide resolved
tests/testsuite/precise_pre_release.rs Outdated Show resolved Hide resolved
tests/testsuite/precise_pre_release.rs Outdated Show resolved Hide resolved
tests/testsuite/precise_pre_release.rs Outdated Show resolved Hide resolved
tests/testsuite/precise_pre_release.rs Outdated Show resolved Hide resolved
This change introduces the feature but does not yet attempt an implementation.
The actual implementation will happen in future PRs

r? @epage
@eopb
Copy link
Contributor Author

eopb commented Jan 14, 2024

Thanks for your review @weihanglo, I've updated my commit with your suggestions

@epage
Copy link
Contributor

epage commented Jan 15, 2024

@bors r+

Looks good to me, that @weihanglo's concerns have been resolved, and if any more crop up, they can be handled as this moves forward.

@bors
Copy link
Collaborator

bors commented Jan 15, 2024

📌 Commit c8ec94c has been approved by epage

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 15, 2024
@bors
Copy link
Collaborator

bors commented Jan 15, 2024

⌛ Testing commit c8ec94c with merge 19eb0f0...

@bors
Copy link
Collaborator

bors commented Jan 15, 2024

☀️ Test successful - checks-actions
Approved by: epage
Pushing 19eb0f0 to master...

1 similar comment
@bors
Copy link
Collaborator

bors commented Jan 15, 2024

☀️ Test successful - checks-actions
Approved by: epage
Pushing 19eb0f0 to master...

@bors bors merged commit 19eb0f0 into rust-lang:master Jan 15, 2024
20 checks passed
@bors
Copy link
Collaborator

bors commented Jan 15, 2024

👀 Test was successful, but fast-forwarding failed: 422 Changes must be made through a pull request.

@eopb eopb deleted the precise-pre-release-flag branch January 15, 2024 19:09
bors added a commit to rust-lang-ci/rust that referenced this pull request Jan 17, 2024
Update cargo

10 commits in 84976cd699f4aea56cb3a90ce3eedeed9e20d5a5..1cff2ee6b92e0ad3f87c44b70b28f788b2528b3c
2024-01-12 15:55:43 +0000 to 2024-01-16 16:56:57 +0000
- doc: add a heading to highlight "How to find features enabled on dependencies" (rust-lang/cargo#13305)
- fix(cargo-update): `--precise` accept arbitrary git revisions (rust-lang/cargo#13250)
- Strip debuginfo when debuginfo is not requested (rust-lang/cargo#13257)
- Update ahash dependency to 0.8.7 (rust-lang/cargo#13301)
- docs: add more links to pkgid spec chapter (rust-lang/cargo#13298)
- fix(metadata): Stabilize id format as PackageIDSpec (rust-lang/cargo#12914)
- Introduce `-Zprecise-pre-release` unstable flag (rust-lang/cargo#13296)
- Delete sentence about parentheses being unsupported in license (rust-lang/cargo#13292)
- Add guidance on setting homepage in manifest (rust-lang/cargo#13293)
- Clarify the function of the test options (rust-lang/cargo#13236)

r? ghost
@ehuss
Copy link
Contributor

ehuss commented Jan 18, 2024

Just FYI, it's a little confusing to me to have documentation for something that isn't implemented. Perhaps in the future there can be a warning indicating it isn't implemented yet?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-documenting-cargo-itself Area: Cargo's documentation A-unstable Area: nightly unstable support S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants