Skip to content

Conversation

natsukagami
Copy link
Contributor

@natsukagami natsukagami commented Jun 16, 2025

Fix #22936

Depends on scalameta/scalameta#4272 (for the generated SemanticsDB to be consumable).

@tgodzik
Copy link
Contributor

tgodzik commented Jun 23, 2025

Thanks for working on this! We simplified a bit the trees in order not to add a new node. Do you need help adjusting the PR according to scalameta/scalameta#4275 ?

@natsukagami
Copy link
Contributor Author

No problem! I will manage to update it, currently a bit busy on something else atm!

@natsukagami natsukagami force-pushed the apply-context-semanticsdb branch from 062adab to b1f223a Compare September 1, 2025 14:20
@natsukagami natsukagami marked this pull request as ready for review September 1, 2025 14:20
@Gedochao Gedochao requested a review from tgodzik September 1, 2025 14:29
Copy link
Contributor

@tgodzik tgodzik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@tgodzik tgodzik enabled auto-merge (squash) September 1, 2025 14:59
@tgodzik tgodzik merged commit 4493b49 into scala:main Sep 1, 2025
44 checks passed
tgodzik pushed a commit to scala/scala3-lts that referenced this pull request Sep 12, 2025
## Fix scala#22936

Depends on scalameta/scalameta#4272 (for the generated SemanticsDB to be
consumable).

- Add `ApplyContextTree` from generated code, generated at
https://github.com/natsukagami/semanticdb-for-scala3/tree/apply-context-tree
- Use ApplyContextTree instead of ApplyTree for context parameter
applications
[Cherry-picked 4493b49]
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.

Add context parameters to SemanticDB Synthetics
2 participants