Skip to content

fix: drop redundant primarykey flags#20

Merged
mplemay merged 1 commit intomainfrom
update-primarykeymixin-id-definition
Mar 7, 2026
Merged

fix: drop redundant primarykey flags#20
mplemay merged 1 commit intomainfrom
update-primarykeymixin-id-definition

Conversation

@mplemay
Copy link
Owner

@mplemay mplemay commented Mar 7, 2026

Summary

  • remove the unique=True and index=True arguments from PrimaryKeyMixin.id so it’s declared as a plain primary key with the existing UUID defaults and init flag preserved
  • update the mixin doc comment to describe only the client/server defaults, no explicit unique/index claim
  • tighten the mixin test expectations to what remains guaranteed (primary key, server default, init exclusion, client UUID default)

Testing

  • Not run (not requested)

for the pull request title use an all-lowercase conventional commit style message (ex: "feat: move dashboard into home route")

@mplemay mplemay merged commit 21aa603 into main Mar 7, 2026
3 checks passed
@mplemay mplemay deleted the update-primarykeymixin-id-definition branch March 7, 2026 01:02
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.

1 participant