Skip to content

Spanner: untangle options vars and traits#8348

Closed
bshaffer wants to merge 1 commit intosnapshot-isolationfrom
snapshot-isolation-untangle-traits
Closed

Spanner: untangle options vars and traits#8348
bshaffer wants to merge 1 commit intosnapshot-isolationfrom
snapshot-isolation-untangle-traits

Conversation

@bshaffer
Copy link
Copy Markdown
Contributor

@bshaffer bshaffer commented May 29, 2025

  • explicitly selecting valid fields
  • better names for variables
  • documenting missing options in phpdoc
  • removing pass-by-reference

TODO:

  • remove remaining pass-by-reference
  • run system tests

  - explicitly selecting valid fields
  - better names for variables
  - documenting missing options in phpdoc
  - removing pass-by-reference
@product-auto-label product-auto-label Bot added the api: spanner Issues related to the Spanner API. label May 29, 2025
@bshaffer
Copy link
Copy Markdown
Contributor Author

bshaffer commented Jul 9, 2025

This PR was merged into #7841, and will be less risky to continue there

@bshaffer bshaffer closed this Jul 9, 2025
@bshaffer bshaffer deleted the snapshot-isolation-untangle-traits branch July 9, 2025 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: spanner Issues related to the Spanner API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant