Skip to content

Use ParamEnv in TraitEnvironment#20728

Merged
ShoyuVanilla merged 4 commits into
rust-lang:masterfrom
jackh726:next-trait-solver-next2
Sep 23, 2025
Merged

Use ParamEnv in TraitEnvironment#20728
ShoyuVanilla merged 4 commits into
rust-lang:masterfrom
jackh726:next-trait-solver-next2

Conversation

@jackh726

Copy link
Copy Markdown
Member

And a couple other chalk -> ns things.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 22, 2025
@rust-cloud-vms rust-cloud-vms Bot force-pushed the next-trait-solver-next2 branch from ee65947 to 34f773f Compare September 23, 2025 00:05

@ShoyuVanilla ShoyuVanilla left a comment

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.

Looks good to me 👍

@ShoyuVanilla ShoyuVanilla added this pull request to the merge queue Sep 23, 2025
498..502 'self': SelectStatement<F, S, D, W, O, LOf, {unknown}, {unknown}>
498..508 'self.order': O
498..515 'self.o...into()': dyn QueryFragment<DB> + '?
498..515 'self.o...into()': dyn QueryFragment<DB> + 'static

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.

BTW this implicit 'static lifetime bound is interesting. I guess this is correct but not sure this is accurate or by accident because our lifetime handling is quite incomplete 😅

Merged via the queue into rust-lang:master with commit 7804a2d Sep 23, 2025
15 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 23, 2025
@jackh726 jackh726 deleted the next-trait-solver-next2 branch September 23, 2025 07:58
@lnicola

lnicola commented Oct 14, 2025

Copy link
Copy Markdown
Member

changelog fixup #20329

Sign up for free to 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.

4 participants