Skip to content

docs(hints): write mostly-unused docs as end-user documentation - #17549

Merged
weihanglo merged 1 commit into
rust-lang:masterfrom
0xPoe:docs-hint-mostly-unused
Oct 26, 2026
Merged

weihanglo merged 1 commit into
rust-lang:masterfrom
0xPoe:docs-hint-mostly-unused

Conversation

@0xPoe

@0xPoe 0xPoe commented Oct 5, 2026 •

Copy link
Copy Markdown
Member

What does this PR try to resolve?

Inspired by #17368 (comment).

So I added the end-user documentation for it.

How to test and review this PR?

r?@ghost

Check the docs.

LLM disclosure: Used for grammar fixes and polishing my words.

@rustbot rustbot added the A-documenting-cargo-itself Area: Cargo's documentation label Oct 5, 2026
@0xPoe 0xPoe changed the title docs(hints): write mostly-unused docs as end-user documentation NOT READY FRO REVIEW: docs(hints): write mostly-unused docs as end-user documentation Oct 5, 2026
@0xPoe
0xPoe force-pushed the docs-hint-mostly-unused branch 2 times, most recently from 26c1687 to 12527af Compare October 5, 2026 13:58
@rustbot

This comment has been minimized.

@rustbot rustbot added the S-waiting-on-author Status: The marked PR is awaiting some action (such as code changes) from the PR author. label Oct 6, 2026
@0xPoe
0xPoe force-pushed the docs-hint-mostly-unused branch 2 times, most recently from 8c32ed1 to b494059 Compare October 26, 2026 09:08
@0xPoe 0xPoe changed the title NOT READY FRO REVIEW: docs(hints): write mostly-unused docs as end-user documentation docs(hints): write mostly-unused docs as end-user documentation Oct 26, 2026
@0xPoe
0xPoe marked this pull request as ready for review October 26, 2026 09:10
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 26, 2026
@0xPoe 0xPoe removed the S-waiting-on-author Status: The marked PR is awaiting some action (such as code changes) from the PR author. label Oct 26, 2026

@0xPoe 0xPoe left a comment •

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

🔢 Self-check (PR reviewed by myself and ready for feedback)

r? @weihanglo

View changes since this review

@0xPoe 0xPoe changed the title docs(hints): write mostly-unused docs as end-user documentation docs(hints): add mostly-unused docs as end-user documentation Oct 26, 2026
@0xPoe 0xPoe changed the title docs(hints): add mostly-unused docs as end-user documentation docs(hints): write mostly-unused docs as end-user documentation Oct 26, 2026
Comment on lines +958 to +961
The valid options are:

* `true`: enabled
* `false`: disabled

@weihanglo weihanglo Oct 26, 2026 •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nit: why do we have an unordered list for profile but not for hint? Which writing style do we prefer and follow the eixsting style?

View changes since the review

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

profiles.md uses this style, so I chose to use it here. manifest.md uses a different style(Or no style😄). I do not think this is a big issue, and I can change either one if you think we should use one style throughout.

Comment thread doc/book/src/reference/unstable.md Outdated
@0xPoe
0xPoe force-pushed the docs-hint-mostly-unused branch from b494059 to df7af8d Compare October 26, 2026 20:05
@0xPoe
0xPoe requested a review from weihanglo October 26, 2026 20:14
@weihanglo
weihanglo enabled auto-merge October 26, 2026 20:38
@weihanglo
weihanglo added this pull request to the merge queue Oct 26, 2026
Merged via the queue into rust-lang:master with commit a4590e4 Oct 26, 2026
111 of 117 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 26, 2026
rust-bors Bot pushed a commit to rust-lang/rust that referenced this pull request Oct 29, 2026
Update cargo submodule

7 commits in cad07cf8e597397fff5d0f6cb79c9f870c0b1dd2..31d3246e0e8d3a80d4a90b54aa48ff180adb408b
2026-10-23 22:04:58 +0000 to 2026-10-27 13:54:39 +0000
- Warn about conflicting proc-macro manifest settings (rust-lang/cargo#17641)
- docs: use latest `resolver` and `edition` in examples (rust-lang/cargo#17644)
- docs(hints): write mostly-unused docs as end-user documentation (rust-lang/cargo#17549)
- refactor(help): embed man pages without a tarball  (rust-lang/cargo#17640)
- refactor(cargo-test-support): remove build.rs (rust-lang/cargo#17638)
- fix: exclude abspath of path deps from `-Cmetadata` hash (rust-lang/cargo#17624)
- use build-rs for buildscripts (rust-lang/cargo#17637)

r? ghost
@rustbot rustbot added this to the 1.101.0 milestone Oct 29, 2026
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants