Skip to content

Conversation

notriddle
Copy link
Contributor

@notriddle notriddle commented Mar 18, 2023

This tweak to the function signature search engine makes things so that, if a type is repeated in the search query, it'll only match if the function actually includes it that many times.

@rustbot
Copy link
Collaborator

rustbot commented Mar 18, 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 Mar 18, 2023
@rustbot
Copy link
Collaborator

rustbot commented Mar 18, 2023

Some changes occurred in HTML/CSS/JS.

cc @GuillaumeGomez, @Folyd, @jsha

@GuillaumeGomez
Copy link
Member

Can you give a bit more explanations about what this is doing please?

@notriddle
Copy link
Contributor Author

@GuillaumeGomez Okay, I added a description to the PR. Does that make sense?

@rust-log-analyzer

This comment has been minimized.

@GuillaumeGomez
Copy link
Member

Yes it does! Thanks for the explanation. Seems like the CI is failing for everyone. Once fixed, r=me.

@notriddle notriddle force-pushed the notriddle/search-bag-semantics branch 2 times, most recently from 09233f9 to 2402b90 Compare March 19, 2023 21:11
@rust-log-analyzer

This comment has been minimized.

@notriddle notriddle force-pushed the notriddle/search-bag-semantics branch from 2402b90 to 19dc4ce Compare March 19, 2023 21:38
@rust-log-analyzer

This comment has been minimized.

@notriddle notriddle force-pushed the notriddle/search-bag-semantics branch from 19dc4ce to 52ef786 Compare March 19, 2023 23:36
@rust-log-analyzer

This comment has been minimized.

This tweak to the function signature search engine makes things so that,
if a type is repeated in the search query, it'll only match if the
function actually includes it that many times.
@notriddle notriddle force-pushed the notriddle/search-bag-semantics branch from 52ef786 to 5451fe7 Compare March 20, 2023 01:19
@notriddle
Copy link
Contributor Author

@bors r=GuillaumeGomez rollup

@bors
Copy link
Collaborator

bors commented Mar 20, 2023

📌 Commit 5451fe7 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 Mar 20, 2023
bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 20, 2023
…iaskrgr

Rollup of 13 pull requests

Successful merges:

 - rust-lang#109249 (Update names/comments for new return-position impl trait in trait lowering strategy)
 - rust-lang#109259 (rustdoc: Fix missing private inlining)
 - rust-lang#109269 (rustdoc: cleanup some intermediate allocs)
 - rust-lang#109301 (fix: fix ICE in `custom-test-frameworks` feature)
 - rust-lang#109319 (Add test for `c_variadic` in rustdoc-json)
 - rust-lang#109323 (Ignore files in .gitignore in mir opt check)
 - rust-lang#109331 (rustdoc: implement bag semantics for function parameter search)
 - rust-lang#109337 (Improve `Iterator::collect_into` documentation)
 - rust-lang#109351 (rustdoc: Remove footnote references from doc summary)
 - rust-lang#109353 (Fix wrong crate name in custom MIR docs)
 - rust-lang#109362 (Split `items` from `-Zmeta-stats` in two.)
 - rust-lang#109370 (fix ClashingExternDeclarations lint ICE)
 - rust-lang#109375 (rustdoc: Fix improper escaping of deprecation reasons)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 272afbe into rust-lang:master Mar 20, 2023
@rustbot rustbot added this to the 1.70.0 milestone Mar 20, 2023
@notriddle notriddle deleted the notriddle/search-bag-semantics branch March 20, 2023 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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