Skip to content

Fix issue #2626 by un-ignoring the mysterious test_leaks, - #5912

Closed
Dretch wants to merge 1 commit into
rust-lang:incomingfrom
Dretch:issue-2626
Closed

Fix issue #2626 by un-ignoring the mysterious test_leaks, #5912
Dretch wants to merge 1 commit into
rust-lang:incomingfrom
Dretch:issue-2626

Conversation

@Dretch

Copy link
Copy Markdown
Contributor

which does currently seem to work on win32 (and linux).

Just mentioning issue #2626 again to make sure github picks it up.

which does currently seem to work on win32 (and linux).
bors added a commit that referenced this pull request Apr 18, 2013
which does currently seem to work on win32 (and linux).
Just mentioning issue #2626 again to make sure github picks it up.
@borsbors closed this Apr 18, 2013
flip1995 pushed a commit to flip1995/rust that referenced this pull request Aug 28, 2020
…nts, r=Manishearth,flip1995
Parse doctests in needless_doctest_main
This switches from text-based search to running the parser to avoid false positives. Inspired by how [rustdoc](https://github.com/rust-lang/rust/blob/3f3250500fe152b5759c21453ba9a9129808d0d8/src/librustdoc/test.rs#L366) handles this and by rust-lang#4729.
cc @llogiq
changelog: Fix multiple false positives in [`needless_doctest_main`].
Fixesrust-lang#5879Fixesrust-lang#4906Fixesrust-lang#5103Fixesrust-lang#4698
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
5910: Fix some typos r=matklad a=SomeoneToIgnore
5912: Remove fixme from inlay_hints.ts r=matklad a=Veetaha
I have reevaluated the fixme and it doesn't seem necessary to pass an array of files
to the inlay hints request.
This will (a) make the request more compilcated (b), make us wait for
inlay hints for `all` active editors resolve at once before rendering and (c)
doesn't seem required because 99% of the time there is a single active editor
in the IDE
Co-authored-by: Kirill Bulatov <mail4score@gmail.com>
Co-authored-by: Veetaha <veetaha2@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@Dretch@bors