Skip to content

Conversation

rekhoff
Copy link
Contributor

@rekhoff rekhoff commented Aug 11, 2025

Description of Changes

Uncommented out code from test, in order to have "delete row by index" test of C# module code ran.
This is the implementation of a fix for issue #2233

API and ABI breaking changes

None

Expected complexity level and risk

1

Testing

  • Ran dotnet run and dotnet test, both returned no errors.

@rekhoff rekhoff self-assigned this Aug 11, 2025
@rekhoff rekhoff requested a review from jdetter August 14, 2025 16:34
@rekhoff rekhoff marked this pull request as ready for review August 14, 2025 16:35
@bfops bfops added the release-any To be landed in any release window label Aug 18, 2025
Copy link
Collaborator

@jdetter jdetter left a comment

Choose a reason for hiding this comment

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

We sync'd on this, lgtm 👍

@rekhoff rekhoff added this pull request to the merge queue Aug 26, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 26, 2025
@rekhoff rekhoff added this pull request to the merge queue Aug 26, 2025
Merged via the queue into master with commit 833f750 Aug 26, 2025
27 checks passed
@rekhoff rekhoff mentioned this pull request Aug 26, 2025
@rekhoff rekhoff deleted the rekhoff/delete-row-by-index branch August 26, 2025 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-any To be landed in any release window
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants