rust icon indicating copy to clipboard operation
rust copied to clipboard

Stabilize `debug_closure_helpers`

Open coolreader18 opened this issue 4 months ago • 14 comments

Resolves rust-lang/rust#117729. The tracking issue still needs an FCP, but I'm hoping that creating a stabilization PR will prompt one.

r? libs-api

coolreader18 avatar Sep 01 '25 17:09 coolreader18

:umbrella: The latest upstream changes (presumably #146636) made this pull request unmergeable. Please resolve the merge conflicts.

bors avatar Sep 27 '25 08:09 bors

Once the FCP is complete, I think it might make sense to merge #145915 first and then merge this on top of that.

coolreader18 avatar Oct 20 '25 17:10 coolreader18

The stabilization PR for fmt::from_fn has merged, so this one is now ready to rebase.

gentle ping @the8472 for review

jmillikin avatar Nov 04 '25 13:11 jmillikin

This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

rustbot avatar Nov 04 '25 18:11 rustbot

:umbrella: The latest upstream changes (presumably #148544) made this pull request unmergeable. Please resolve the merge conflicts.

bors avatar Nov 06 '25 01:11 bors

This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

rustbot avatar Nov 06 '25 02:11 rustbot

The job x86_64-gnu-tools failed! Check out the build log: (web) (plain enhanced) (plain)

Click to see the possible cause of the failure (guessed by this bot)
[WARNING] `tests/rustdoc-gui/search-result-display.goml` line 39: Delta is 0 for "x", maybe try to use `compare-elements-position` instead?

======== tests/rustdoc-gui/source-code-wrapping.goml ========

[ERROR] `tests/rustdoc-gui/source-code-wrapping.goml` line 30: Error: Navigating frame was detached: for command `go-to: "file://" + |DOC_PATH| + "/test_docs/trait_bounds/index.html"`
`tests/rustdoc-gui/source-code-wrapping.goml` source-code-wrapping output:
Protocol error: Connection closed. Most likely the page has been closed.
stack: Error: Protocol error: Connection closed. Most likely the page has been closed.
    at assert (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/puppeteer-core/lib/cjs/puppeteer/util/assert.js:18:15)
    at CdpPage.close (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/puppeteer-core/lib/cjs/puppeteer/cdp/Page.js:814:36)
    at async runAllCommands (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/browser-ui-test/src/index.js:433:13)
    at async innerRunTestCode (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/browser-ui-test/src/index.js:696:21)
    at async innerRunTests (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/browser-ui-test/src/index.js:633:17)
    at async runTests (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/browser-ui-test/src/index.js:824:27)


======== tests/rustdoc-gui/source-code-wrapping.goml ========

[ERROR] `tests/rustdoc-gui/source-code-wrapping.goml` line 30: Error: Navigating frame was detached: for command `go-to: "file://" + |DOC_PATH| + "/test_docs/trait_bounds/index.html"`
`tests/rustdoc-gui/source-code-wrapping.goml` source-code-wrapping output:
Protocol error: Connection closed. Most likely the page has been closed.
stack: Error: Protocol error: Connection closed. Most likely the page has been closed.
    at assert (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/puppeteer-core/lib/cjs/puppeteer/util/assert.js:18:15)
    at CdpPage.close (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/puppeteer-core/lib/cjs/puppeteer/cdp/Page.js:814:36)
    at async runAllCommands (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/browser-ui-test/src/index.js:433:13)
    at async innerRunTestCode (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/browser-ui-test/src/index.js:696:21)
    at async innerRunTests (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/browser-ui-test/src/index.js:633:17)
    at async runTests (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/browser-ui-test/src/index.js:824:27)



<= doc-ui tests done: 142 succeeded, 1 failed, 0 filtered out

rust-log-analyzer avatar Nov 06 '25 03:11 rust-log-analyzer

@bors retry

jmillikin avatar Nov 06 '25 06:11 jmillikin

@jmillikin: :key: Insufficient privileges: not in try users

bors avatar Nov 06 '25 06:11 bors

@the8472 gentle ping?

jmillikin avatar Nov 22 '25 04:11 jmillikin

r? libs-api

jmillikin avatar Dec 01 '25 02:12 jmillikin

:umbrella: The latest upstream changes (presumably #149704) made this pull request unmergeable. Please resolve the merge conflicts.

bors avatar Dec 06 '25 16:12 bors

r? tgross35

Fyi you can r? libs rather than libs-api for stabilization PRs, libs does more reviews. libs-api just needs to do the FCP.

The diff here LGTM, but I'd just like to make sure we have room to improve on what Hanna mentioned.

tgross35 avatar Dec 06 '25 19:12 tgross35

Also this needs a rebase, of course

tgross35 avatar Dec 06 '25 19:12 tgross35

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

rustbot avatar Dec 08 '25 18:12 rustbot

:umbrella: The latest upstream changes (presumably #149941) made this pull request unmergeable. Please resolve the merge conflicts.

bors avatar Dec 13 '25 11:12 bors

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

rustbot avatar Dec 15 '25 01:12 rustbot