Skip to content

CSHARP-4931 Add tests for SortDefinitionExtensions#1935

Merged
kyra-rk merged 4 commits intomongodb:mainfrom
kyra-rk:csharp-4931
Apr 8, 2026
Merged

CSHARP-4931 Add tests for SortDefinitionExtensions#1935
kyra-rk merged 4 commits intomongodb:mainfrom
kyra-rk:csharp-4931

Conversation

@kyra-rk
Copy link
Copy Markdown
Contributor

@kyra-rk kyra-rk commented Apr 2, 2026

No description provided.

@kyra-rk kyra-rk added the improvement Optimizations or refactoring (no new features or fixes). label Apr 2, 2026
@kyra-rk kyra-rk requested a review from papafe April 2, 2026 15:51
@kyra-rk kyra-rk marked this pull request as ready for review April 2, 2026 15:54
@kyra-rk kyra-rk requested a review from a team as a code owner April 2, 2026 15:54
Copy link
Copy Markdown
Contributor

@papafe papafe left a comment

Choose a reason for hiding this comment

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

Looks good, just a small comment.

}

[Fact]
public void Ascending_typed_using_extension_methods()
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ascending_Typed.... (just to keep consistency)

@kyra-rk kyra-rk requested a review from papafe April 8, 2026 14:58
Copy link
Copy Markdown
Contributor

@papafe papafe left a comment

Choose a reason for hiding this comment

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

LGTM. Let's just wait for evergreen before merging.

@kyra-rk kyra-rk merged commit 457c80e into mongodb:main Apr 8, 2026
31 of 33 checks passed
@papafe papafe added maintenance Non-code maintenance (deps, docs, configs, etc.). chore Non–user-facing code changes (tests, build scripts, etc.). and removed improvement Optimizations or refactoring (no new features or fixes). maintenance Non-code maintenance (deps, docs, configs, etc.). labels Apr 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Non–user-facing code changes (tests, build scripts, etc.).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants