Skip to content

Update expected test results in contextParameters test - #3218

Merged
jaschdoc merged 1 commit into
mainfrom
fix-ctx-param-test
Sep 24, 2026
Merged

jaschdoc merged 1 commit into
mainfrom
fix-ctx-param-test

Conversation

@jaschdoc

Copy link
Copy Markdown
Collaborator

The expected results are now different based on the EXPECT NEXT feature toggle.

Related to #2472

@jaschdoc
jaschdoc marked this pull request as ready for review September 22, 2026 08:04
@jaschdoc
jaschdoc enabled auto-merge September 22, 2026 08:04
@jaschdoc
jaschdoc changed the base branch from main to context-parameter-impl September 22, 2026 13:36
@jaschdoc
jaschdoc requested a review from hfmehmed as a code owner September 22, 2026 13:36
auto-merge was automatically disabled September 22, 2026 13:36

Merge commits are not allowed on this repository

hfmehmed
hfmehmed previously approved these changes Sep 22, 2026
@jaschdoc
jaschdoc marked this pull request as draft September 22, 2026 13:38
@jaschdoc
jaschdoc force-pushed the context-parameter-impl branch from 8e65db9 to ed259bc Compare September 22, 2026 13:41
@jaschdoc
jaschdoc force-pushed the fix-ctx-param-test branch 2 times, most recently from 5ee71bc to 28bdfd1 Compare September 22, 2026 13:47
@jaschdoc
jaschdoc changed the base branch from context-parameter-impl to main September 22, 2026 13:47
@jaschdoc
jaschdoc dismissed hfmehmed’s stale review September 22, 2026 13:47

The base branch was changed.

The expected results are now different based on the EXPECT NEXT feature toggle,
allowing the baseline version to ignore context parameters.
@jaschdoc
jaschdoc marked this pull request as ready for review September 22, 2026 13:52
@jaschdoc
jaschdoc enabled auto-merge September 22, 2026 14:32
@jaschdoc
jaschdoc added this pull request to the merge queue Sep 24, 2026
Merged via the queue into main with commit 41d6ce3 Sep 24, 2026
28 checks passed
@jaschdoc
jaschdoc deleted the fix-ctx-param-test branch September 24, 2026 10:32
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.

3 participants