Skip to content

make UserInfo likes_count, video_count, follower_count, and following_count, as well as Comment video_id, parent_comment_id, like_count, reply_count bigintegers. since they can exceed 32 bit int#109

Merged
macpd merged 6 commits intomainfrom
fix/big_int_user_info
Mar 28, 2025
Merged

make UserInfo likes_count, video_count, follower_count, and following_count, as well as Comment video_id, parent_comment_id, like_count, reply_count bigintegers. since they can exceed 32 bit int#109
macpd merged 6 commits intomainfrom
fix/big_int_user_info

Conversation

@macpd
Copy link
Copy Markdown
Contributor

@macpd macpd commented Mar 27, 2025

also, fix alembic dependency on black (used in hook to format revision files)

macpd added 4 commits March 27, 2025 16:28
…_count, as well as Comment video_id, parent_comment_id, like_count, reply_count bigintegers. since they can exceed 32 bit int
…count, and following_count, as well as Comment video_id, parent_comment_id, like_count, reply_count DB schema to bigintegers.
@macpd macpd changed the title Fix/big int user info make UserInfo likes_count, video_count, follower_count, and following_count, as well as Comment video_id, parent_comment_id, like_count, reply_count bigintegers. since they can exceed 32 bit int Mar 27, 2025
@macpd macpd requested a review from BrunoGomesCoelho March 28, 2025 13:53
Copy link
Copy Markdown
Collaborator

@BrunoGomesCoelho BrunoGomesCoelho left a comment

Choose a reason for hiding this comment

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

Thanks Paul! Added a commit on top after running the hatch fmt

@macpd macpd merged commit a104a59 into main Mar 28, 2025
1 check passed
@macpd macpd deleted the fix/big_int_user_info branch March 28, 2025 19:24
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.

2 participants