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

docs(contrib): Describe how to add a new package #12878

Merged
merged 1 commit into from
Oct 25, 2023
Merged

Conversation

epage
Copy link
Contributor

@epage epage commented Oct 25, 2023

@Muscraft is considering adding a crate and we felt it was best to not just answer the question of how to do so but to document it!

@Muscraft is considering adding a crate and we felt it was best to not
just answer the question of how to do so but to document it!
@rustbot
Copy link
Collaborator

rustbot commented Oct 25, 2023

r? @weihanglo

(rustbot has picked a reviewer for you, use r? to override)

@rustbot rustbot added A-documenting-cargo-itself Area: Cargo's documentation S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 25, 2023
src/doc/contrib/src/implementation/packages.md Outdated Show resolved Hide resolved
- Add it to `.cargo/config.toml`s `[alias]` table
- Mark `package.publish = false`

If published, add the package to `publish.py`, in dependency order
Copy link
Member

Choose a reason for hiding this comment

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

It's unclear whether a package is required to add to the list. Can we expand this a bit? Something like

Suggested change
If published, add the package to `publish.py`, in dependency order
If published, add the package to the [`TO_PUBLISH` list in `publish.py`](https://github.com/rust-lang/cargo/blob/9bf67a1026c023e2789e867d8b88dc6fc181f357/publish.py#L18-L28), in dependency order.
Packages in `TO_PUBLISH` will be published automatically along with each new releases of Rust.
Note that every package used by `cargo` the library is required to be available on crates.io, but not necessary to be in the `TO_PUBLISH` list.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was wanting to hold off on getting into that level of detail to be more resilient against change. I have updated it. Is the new version sufficient?

src/doc/contrib/src/implementation/packages.md Outdated Show resolved Hide resolved
@epage epage force-pushed the contrib branch 2 times, most recently from ea592ba to 9a36c04 Compare October 25, 2023 17:40
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.

I am pretty okay with this doc update 👍🏾.
Thank you.

@weihanglo
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Oct 25, 2023

📌 Commit 9a36c04 has been approved by weihanglo

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 Oct 25, 2023
@bors
Copy link
Contributor

bors commented Oct 25, 2023

⌛ Testing commit 9a36c04 with merge 03e7022...

@bors
Copy link
Contributor

bors commented Oct 25, 2023

☀️ Test successful - checks-actions
Approved by: weihanglo
Pushing 03e7022 to master...

@bors bors merged commit 03e7022 into rust-lang:master Oct 25, 2023
20 checks passed
bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 28, 2023
Update cargo

8 commits in df3509237935f9418351b77803df7bc05c009b3d..708383d620e183a9ece69b8fe930c411d83dee27
2023-10-24 23:09:01 +0000 to 2023-10-27 21:09:26 +0000
- feat(doc): Print the generated docs links (rust-lang/cargo#12859)
- feat(toml): Allow version-less manifests (rust-lang/cargo#12786)
- Remove outdated option to `-Zcheck-cfg` warnings (rust-lang/cargo#12884)
- Remove duplicate binaries during install (rust-lang/cargo#12868)
- refactor(shell): Write at once rather than in fragments (rust-lang/cargo#12880)
- docs(ref): Link to docs.rs metadata table (rust-lang/cargo#12879)
- docs(contrib): Describe how to add a new package (rust-lang/cargo#12878)
- move up looking at index summary enum (rust-lang/cargo#12749)

r? ghost
@epage epage deleted the contrib branch October 28, 2023 20:59
@ehuss ehuss added this to the 1.75.0 milestone Nov 6, 2023
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 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.

5 participants