Skip to content

Conversation

@lesaltzm
Copy link
Contributor

@lesaltzm lesaltzm commented Oct 3, 2025

Some properties that depend on ThisItem scope will have significant performance costs to async operations. We've discussed and landed on restricting the set of things that can be accessed from those properties to just sync functions, the thisitem scope, enums, and operators.

This PR is a first pass at this, but it won't be immediately exposed to customers, so we'll iterate on it.

@lesaltzm lesaltzm requested a review from a team as a code owner October 3, 2025 20:44
@CarlosFigueiraMSFT
Copy link
Contributor

        return new BindingConfig(

Need to pass the enforceSimpleExpressions parameter to the .ctor when cloning.


Refers to: src/libraries/Microsoft.PowerFx.Core/Binding/BindingConfig.cs:57 in 8f2a273. [](commit_id = 8f2a273, deletion_comment = False)

@jas-valgotar
Copy link
Contributor

✅ No public API change.

@lesaltzm lesaltzm enabled auto-merge (squash) October 7, 2025 16:39
Copy link
Contributor

@CarlosFigueiraMSFT CarlosFigueiraMSFT left a comment

Choose a reason for hiding this comment

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

:shipit:

@jas-valgotar
Copy link
Contributor

✅ No public API change.

@lesaltzm lesaltzm merged commit 0330bc1 into main Oct 7, 2025
4 checks passed
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