Skip to content

Conversation

@nbbeeken
Copy link
Contributor

Please complete the following before merging:

  • Update changelog.
  • Test changes in at least one language driver.
  • Test these changes against all server versions and topologies (including standalone, replica set, sharded
    clusters, and serverless).

The changes don't require drivers to change their tests they just would've found these details out themselves when they went to implment the test.

@nbbeeken nbbeeken requested a review from a team as a code owner March 27, 2025 17:09
@nbbeeken nbbeeken requested review from durran and removed request for a team March 27, 2025 17:09
@nbbeeken nbbeeken force-pushed the clarify-txn-prose-requirements branch from cfb45b1 to 66c1085 Compare March 27, 2025 17:12
@baileympearson
Copy link
Contributor

@dmitry-lukyanov - I added you as a reviewer because Durran is OOO. This PR includes fixes a few issues we noticed with this prose test when we implemented it during spec fest.

@nbbeeken nbbeeken merged commit f354960 into master Mar 28, 2025
5 checks passed
@nbbeeken nbbeeken deleted the clarify-txn-prose-requirements branch March 28, 2025 15: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.

4 participants