Skip to content

Conversation

@notriddle
Copy link
Contributor

This feature extends rustdoc to support the syntax that most users will naturally attempt to use to search for diverging functions. Part of #60485

It's already possible to do this search with primitive:never, but that's not what the Rust language itself uses, so nobody will try it if they aren't told or helped along.

@rustbot
Copy link
Collaborator

rustbot commented Jun 12, 2023

r? @GuillaumeGomez

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

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. labels Jun 12, 2023
@rustbot
Copy link
Collaborator

rustbot commented Jun 12, 2023

Some changes occurred in HTML/CSS/JS.

cc @GuillaumeGomez, @Folyd, @jsha

@rust-log-analyzer

This comment has been minimized.

This feature extends rustdoc to support the syntax that most users will
naturally attempt to use to search for diverging functions.
Part of rust-lang#60485

It's already possible to do this search with `primitive:never`, but
that's not what the Rust language itself uses, so nobody will try it if
they aren't told or helped along.
@notriddle notriddle force-pushed the notriddle/never-search branch from 7673af0 to db277f5 Compare June 13, 2023 00:30
@GuillaumeGomez
Copy link
Member

Just wondering: is it really useful to add such checks for this type specifically? It's a doc alias after all...

@notriddle
Copy link
Contributor Author

It definitely can’t just be a doc(alias), since those only apply to the crate that declares them, and we want this to work in crates other than the standard library.

@GuillaumeGomez
Copy link
Member

Ok I see what I missed: the never type is a type and can be present in query as such with this PR. Thanks for the clarification!

@GuillaumeGomez
Copy link
Member

Thanks!

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Jun 13, 2023

📌 Commit db277f5 has been approved by GuillaumeGomez

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 Jun 13, 2023
@GuillaumeGomez GuillaumeGomez added A-type-based-search Area: Searching rustdoc pages using type signatures A-rustdoc-search Area: Rustdoc's search feature labels Jun 13, 2023
bors added a commit to rust-lang-ci/rust that referenced this pull request Jun 14, 2023
…iaskrgr

Rollup of 5 pull requests

Successful merges:

 - rust-lang#112197 (Erase regions even if normalization fails in writeback (in new solver))
 - rust-lang#112495 (fix(resolve): update shadowed_glob more precision)
 - rust-lang#112520 (Fix the overflow issue for transmute_generic_consts)
 - rust-lang#112571 (rustdoc-search: search never type with `!`)
 - rust-lang#112581 ([rustdoc] Fix URL encoding of % sign)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit e3021b4 into rust-lang:master Jun 14, 2023
@rustbot rustbot added this to the 1.72.0 milestone Jun 14, 2023
@notriddle notriddle deleted the notriddle/never-search branch June 14, 2023 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-rustdoc-search Area: Rustdoc's search feature A-type-based-search Area: Searching rustdoc pages using type signatures S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants