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

feat: stabilize cargo update --precise <yanked> #13974

Merged
merged 1 commit into from
May 29, 2024

Conversation

weihanglo
Copy link
Member

What does this PR try to resolve?

Stabilize cargo update --precise <yanked>.

The cargo team has discussed the stabilization in the meeting today.
The interface of this is quite small and not as controversial as --precise <pre-release>,
since there is no version requirement operators involved.
We'd like to move forward and stabilize this part.

Note that --precise <yanked> allows using yanked version only for the specified package,
We leave the cascading allowing yanked versions (e.g. #4225 (comment)) as a future extension.

How should we test and review this PR?

Check if any adjustment needed for warnings and CLI help text.

Additional information

cc #4225.

@weihanglo weihanglo added the T-cargo Team: Cargo label May 28, 2024
@rustbot
Copy link
Collaborator

rustbot commented May 28, 2024

r? @ehuss

rustbot has assigned @ehuss.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added A-cli-help Area: built-in command-line help A-documenting-cargo-itself Area: Cargo's documentation A-registries Area: registries S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 28, 2024
@weihanglo
Copy link
Member Author

@rfcbot fcp merge

Although we discussed this earlier today as a team, it's better to have a record here.

@rfcbot
Copy link
Collaborator

rfcbot commented May 28, 2024

Team member @weihanglo has proposed to merge this. The next step is review by the rest of the tagged team members:

No concerns currently listed.

Once a majority of reviewers approve (and at most 2 approvals are outstanding), this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up!

See this document for info about what commands tagged team members can give me.

@rfcbot rfcbot added proposed-final-comment-period An FCP proposal has started, but not yet signed off. disposition-merge FCP with intent to merge final-comment-period FCP — a period for last comments before action is taken and removed proposed-final-comment-period An FCP proposal has started, but not yet signed off. labels May 28, 2024
@rfcbot
Copy link
Collaborator

rfcbot commented May 29, 2024

🔔 This is now entering its final comment period, as per the review above. 🔔

@arlosi
Copy link
Contributor

arlosi commented May 29, 2024

Thanks!

@bors r+

@bors
Copy link
Collaborator

bors commented May 29, 2024

📌 Commit a076f79 has been approved by arlosi

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 May 29, 2024
@bors
Copy link
Collaborator

bors commented May 29, 2024

⌛ Testing commit a076f79 with merge c045c8f...

bors added a commit that referenced this pull request May 29, 2024
feat: stabilize `cargo update --precise <yanked>`

### What does this PR try to resolve?

Stabilize `cargo update --precise <yanked>`.

The cargo team has discussed the stabilization in the meeting today.
The interface of this is quite small and not as controversial as `--precise <pre-release>`,
since there is no version requirement operators involved.
We'd like to move forward and stabilize this part.

Note that `--precise <yanked>` allows using yanked version only for the specified package,
We leave the cascading allowing yanked versions (e.g. <#4225 (comment)>) as a future extension.

### How should we test and review this PR?

Check if any adjustment needed for warnings and CLI help text.

### Additional information

cc <#4225>.
@bors
Copy link
Collaborator

bors commented May 29, 2024

💔 Test failed - checks-actions

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

#0 building with "default" instance using docker driver

#1 [internal] load .dockerignore
#1 transferring context: 2B done
#1 DONE 0.0s

#2 [internal] load build definition from Dockerfile
#2 transferring dockerfile: 852B done
#2 DONE 0.0s

#3 [internal] load metadata for docker.io/library/alpine:3.19
#3 ERROR: pulling from host registry-1.docker.io failed with status code [manifests 3.19]: 503 Service Unavailable
------
 > [internal] load metadata for docker.io/library/alpine:3.19:
------
Dockerfile:1
--------------------
   1 | >>> FROM alpine:3.19
   2 |     
   3 |     RUN apk add --no-cache openssh git
--------------------
ERROR: failed to solve: alpine:3.19: pulling from host registry-1.docker.io failed with status code [manifests 3.19]: 503 Service Unavailable


---- ssh::ssh_key_in_config stdout ----
SSH_AGENT_PID=97097
thread 'ssh::ssh_key_in_config' panicked at crates/cargo-test-support/src/containers.rs:99:26:

Network error.

@bors retry

@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 May 29, 2024
@bors
Copy link
Collaborator

bors commented May 29, 2024

⌛ Testing commit a076f79 with merge 38b39a7...

bors added a commit that referenced this pull request May 29, 2024
feat: stabilize `cargo update --precise <yanked>`

### What does this PR try to resolve?

Stabilize `cargo update --precise <yanked>`.

The cargo team has discussed the stabilization in the meeting today.
The interface of this is quite small and not as controversial as `--precise <pre-release>`,
since there is no version requirement operators involved.
We'd like to move forward and stabilize this part.

Note that `--precise <yanked>` allows using yanked version only for the specified package,
We leave the cascading allowing yanked versions (e.g. <#4225 (comment)>) as a future extension.

### How should we test and review this PR?

Check if any adjustment needed for warnings and CLI help text.

### Additional information

cc <#4225>.
@bors
Copy link
Collaborator

bors commented May 29, 2024

💔 Test failed - checks-actions

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

@bors retry

@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 May 29, 2024
@bors
Copy link
Collaborator

bors commented May 29, 2024

⌛ Testing commit a076f79 with merge 2c8d176...

@bors
Copy link
Collaborator

bors commented May 29, 2024

☀️ Test successful - checks-actions
Approved by: arlosi
Pushing 2c8d176 to master...

@bors bors merged commit 2c8d176 into rust-lang:master May 29, 2024
21 checks passed
@weihanglo weihanglo deleted the precise-yank branch May 29, 2024 19:57
bors added a commit to rust-lang-ci/rust that referenced this pull request Jun 1, 2024
Update cargo

9 commits in 431db31d0dbeda320caf8ef8535ea48eb3093407..7a6fad0984d28c8330974636972aa296b67c4513
2024-05-28 18:17:31 +0000 to 2024-05-31 22:26:03 +0000
- fix(config): Ensure `--config net.git-fetch-with-cli=true` is respected (rust-lang/cargo#13992)
- Fix libcurl proxy documentation link (rust-lang/cargo#13990)
- fix(new): Dont say were adding to a workspace when a regular package is in root (rust-lang/cargo#13987)
- fix: adjust custom err from cert-check due to libgit2 1.8 change (rust-lang/cargo#13970)
- fix(toml): Ensure targets are in a deterministic order (rust-lang/cargo#13989)
- doc(cargo-package): explain no guarantee of vcs provenance (rust-lang/cargo#13984)
- chore: fix some comments (rust-lang/cargo#13982)
- feat: stabilize `cargo update --precise &lt;yanked&gt;` (rust-lang/cargo#13974)
- Update openssl-src to 111.28.2+1.1.1w (rust-lang/cargo#13976)

r? ghost
@rustbot rustbot added this to the 1.80.0 milestone Jun 1, 2024
@rfcbot rfcbot added finished-final-comment-period FCP complete to-announce and removed final-comment-period FCP — a period for last comments before action is taken labels Jun 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-cli-help Area: built-in command-line help A-documenting-cargo-itself Area: Cargo's documentation A-registries Area: registries disposition-merge FCP with intent to merge finished-final-comment-period FCP complete S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-cargo Team: Cargo to-announce
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants