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

test: support publish package with a public field. #13245

Merged
merged 1 commit into from
Jan 3, 2024

Conversation

linyihai
Copy link
Contributor

@linyihai linyihai commented Jan 3, 2024

What does this PR try to resolve?

This PR add a public alike method to support add a dependency as public/private,

Package::new("foo", "0.1.0")
        .cargo_feature("public-dependency").add_dep(Dependency::new("bar", "0.1.0").public(true))

and then get it from registry in test.

This PR was seperated from the #13183.

How should we test and review this PR?

You can review on per commit.

After running the test case publish_package_with_public_dependency, you can check a "public" field in ./target/tmp/cit/t0/registry/3/b/bar.

Additional information

r? epage

@rustbot rustbot added A-testing-cargo-itself Area: cargo's tests S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 3, 2024
@epage
Copy link
Contributor

epage commented Jan 3, 2024

@bors r

@bors
Copy link
Collaborator

bors commented Jan 3, 2024

📌 Commit 35d5fae 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 3, 2024
@bors
Copy link
Collaborator

bors commented Jan 3, 2024

⌛ Testing commit 35d5fae with merge 04398ba...

bors added a commit that referenced this pull request Jan 3, 2024
test: support publish package with a `public` field.

### What does this PR try to resolve?

This PR add a `public` alike method to support add a dependency as public/private,
```
Package::new("foo", "0.1.0")
        .cargo_feature("public-dependency").add_dep(Dependency::new("bar", "0.1.0").public(true))
```

and then get it from registry in test.

This PR was seperated from the #13183.

### How should we test and review this PR?
You can review on per commit.

After running the test case `publish_package_with_public_dependency`,  you can check a "public" field in `./target/tmp/cit/t0/registry/3/b/bar`.

### Additional information
r? epage
@epage
Copy link
Contributor

epage commented Jan 3, 2024

@bors r-

@bors bors added S-waiting-on-author Status: The marked PR is awaiting some action (such as code changes) from the PR author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jan 3, 2024
@epage
Copy link
Contributor

epage commented Jan 3, 2024

Sorry, wanted to double check some things first.

@bors r

@bors
Copy link
Collaborator

bors commented Jan 3, 2024

📌 Commit 35d5fae 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-author Status: The marked PR is awaiting some action (such as code changes) from the PR author. labels Jan 3, 2024
@bors
Copy link
Collaborator

bors commented Jan 3, 2024

⌛ Testing commit 35d5fae with merge 8de3343...

@bors
Copy link
Collaborator

bors commented Jan 3, 2024

☀️ Test successful - checks-actions
Approved by: epage
Pushing 8de3343 to master...

@bors bors merged commit 8de3343 into rust-lang:master Jan 3, 2024
20 checks passed
bors added a commit to rust-lang-ci/rust that referenced this pull request Jan 6, 2024
Update cargo

10 commits in add15366eaf3f3eb84717d3b8b71902ca36a7c84..2ce45605d9db521b5fd6c1211ce8de6055fdb24e
2024-01-02 03:24:42  0000 to 2024-01-04 18:04:13  0000
- feat(embedded): Add prefix-char frontmatter syntax support (rust-lang/cargo#13247)
- Update dependency handlebars to v5 for mdman. (rust-lang/cargo#13249)
- Deprecate rustc plugin support in cargo (rust-lang/cargo#13248)
- feat(embedded): Add multiple experimental manifest syntaxes (rust-lang/cargo#13241)
- chore: update auto labels (rust-lang/cargo#13244)
- test: support publish package with a `public` field. (rust-lang/cargo#13245)
- `cargo fix`: Call rustc fewer times. (rust-lang/cargo#13243)
- chore: tracing be compat with rustc_log (rust-lang/cargo#13239)
- chore(deps): update compatible (rust-lang/cargo#13227)
- Contrib: Fix team HackMD links (rust-lang/cargo#13237)

r? ghost
@rustbot rustbot added this to the 1.77.0 milestone Jan 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testing-cargo-itself Area: cargo's tests 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.

None yet

4 participants