Skip to content

Added projection test cases for positional $#240

Merged
eerxuan merged 2 commits into
documentdb:mainfrom
vic-tsang:projection/positional/tests
Jun 1, 2026
Merged

Added projection test cases for positional $#240
eerxuan merged 2 commits into
documentdb:mainfrom
vic-tsang:projection/positional/tests

Conversation

@vic-tsang
Copy link
Copy Markdown
Collaborator

This PR contains:

  • projection test cases for positional $

Ref:

@vic-tsang vic-tsang requested a review from a team as a code owner May 26, 2026 17:57
@documentdb-triage-tool documentdb-triage-tool Bot added compatibility test Compatibility test related enhancement New feature or request labels May 26, 2026
@documentdb-triage-tool
Copy link
Copy Markdown

🤖 Auto-triaged by documentdb-triage-tool.

Applied: compatibility test, enhancement
Project fields suggested: Component test-coverage · Priority P2 · Effort L · Status Needs Review
Confidence: 0.85 (mixed)

Reasoning

component from path globs (test-coverage, test-framework); effort from diff stats (1378+0 LOC, 9 files); LLM: Adds new test cases for positional $ projection operator, expanding compatibility test coverage.

If a label is wrong, remove it manually and ping @patty-chow so the rules can be tuned. The bot will not re-label items that already have component labels.

@vic-tsang vic-tsang closed this May 29, 2026
@vic-tsang vic-tsang force-pushed the projection/positional/tests branch from f52d928 to 1f49420 Compare May 29, 2026 21:44
Signed-off-by: Victor Tsang <vitsangp@amazon.com>
@vic-tsang
Copy link
Copy Markdown
Collaborator Author

added comments and fixed conflicts

@vic-tsang vic-tsang reopened this May 29, 2026
@vic-tsang vic-tsang requested a review from fanyangv May 29, 2026 21:51
@eerxuan eerxuan merged commit fa76a83 into documentdb:main Jun 1, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

compatibility test Compatibility test related enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants