Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: Make database query objects move-only. #263

Merged
merged 1 commit into from
Jan 10, 2025

Conversation

iphydf
Copy link
Member

@iphydf iphydf commented Dec 20, 2024


This change is Reviewable

@iphydf iphydf added this to the v1.18.1 milestone Dec 20, 2024
@github-actions github-actions bot added the refactor Refactoring production code, eg. renaming a variable, not affecting semantics label Dec 20, 2024
@iphydf iphydf force-pushed the db-impl branch 6 times, most recently from c8798fd to 2fab4af Compare December 22, 2024 00:49
@iphydf iphydf force-pushed the db-impl branch 2 times, most recently from 848904b to 290b747 Compare January 2, 2025 00:32
@iphydf iphydf modified the milestones: v1.18.1, v1.18.2 Jan 3, 2025
@iphydf iphydf force-pushed the db-impl branch 8 times, most recently from 8197fb9 to 4416060 Compare January 9, 2025 11:04
@iphydf iphydf changed the title refactor: Hide database implementation behind an interface. refactor: Make database query objects move-only. Jan 9, 2025
@iphydf iphydf marked this pull request as draft January 9, 2025 11:08
@iphydf iphydf force-pushed the db-impl branch 2 times, most recently from cd446e2 to 2cb89ee Compare January 9, 2025 12:05
@iphydf iphydf marked this pull request as ready for review January 9, 2025 12:07
@toktok-releaser toktok-releaser merged commit 2a9b498 into TokTok:master Jan 10, 2025
32 checks passed
@iphydf iphydf deleted the db-impl branch January 10, 2025 23:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Refactoring production code, eg. renaming a variable, not affecting semantics
Development

Successfully merging this pull request may close these issues.

3 participants