Preserve sources of variable values #3077
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
leebyron
force-pushed
the
variable-value-sources
branch
2 times, most recently
from
a271cab
to
fdb733e
Compare
leebyron
force-pushed
the
default-value-literals
branch
from
2ee1da8
to
330f01a
Compare
leebyron
force-pushed
the
variable-value-sources
branch
from
fdb733e
to
6cecfdd
Compare
leebyron
force-pushed
the
default-value-literals
branch
from
330f01a
to
05db9bd
Compare
leebyron
force-pushed
the
variable-value-sources
branch
from
6cecfdd
to
abdec72
Compare
leebyron
force-pushed
the
default-value-literals
branch
from
05db9bd
to
6c79c53
Compare
leebyron
force-pushed
the
variable-value-sources
branch
from
abdec72
to
7e9ce87
Compare
leebyron
force-pushed
the
default-value-literals
branch
3 times, most recently
from
13e3148
to
780cec2
Compare
leebyron
force-pushed
the
variable-value-sources
branch
from
7e9ce87
to
ece9e18
Compare
leebyron
force-pushed
the
default-value-literals
branch
from
780cec2
to
5686d3e
Compare
leebyron
force-pushed
the
variable-value-sources
branch
2 times, most recently
from
f34abdd
to
4b8c50a
Compare
leebyron
force-pushed
the
default-value-literals
branch
from
5686d3e
to
705f154
Compare
leebyron
force-pushed
the
variable-value-sources
branch
from
4b8c50a
to
b967e71
Compare
leebyron
force-pushed
the
default-value-literals
branch
from
705f154
to
dd31191
Compare
leebyron
force-pushed
the
variable-value-sources
branch
from
b967e71
to
14d3f8d
Compare
leebyron
force-pushed
the
default-value-literals
branch
2 times, most recently
from
448367e
to
e2002dc
Compare
leebyron
force-pushed
the
variable-value-sources
branch
from
14d3f8d
to
25ec49a
Compare
leebyron
force-pushed
the
default-value-literals
branch
from
e2002dc
to
5601fd4
Compare
leebyron
force-pushed
the
variable-value-sources
branch
from
25ec49a
to
0c65c4d
Compare
leebyron
force-pushed
the
default-value-literals
branch
from
5601fd4
to
3110ca1
Compare
leebyron
force-pushed
the
variable-value-sources
branch
from
0c65c4d
to
cbf56f0
Compare
leebyron
force-pushed
the
default-value-literals
branch
from
3110ca1
to
f1a14c6
Compare
By way of introducing type `VariableValues`, allows `getVariableValues` to return both the coerced values as well as the original sources, which are then made available in `ExecutionContext`.
leebyron
force-pushed
the
variable-value-sources
branch
from
cbf56f0
to
6fea731
Compare
This was referenced Dec 31, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Depends on #3074
By way of introducing type
VariableValues, allowsgetVariableValuesto return both the coerced values as well as the original sources, which are then made available inExecutionContext.While variable sources are not used directly here, they're used directly in #3065. This PR is pulled out as a pre-req to aid review