Skip to content

Conversation

@psharma-1909
Copy link

No description provided.

@psharma-1909 psharma-1909 self-assigned this May 30, 2025
Comment on lines 2274 to 2276
Copy link
Collaborator

Choose a reason for hiding this comment

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

do we need these prints?

Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: new line

Copy link
Collaborator

@pmishchenko-ua pmishchenko-ua May 30, 2025

Choose a reason for hiding this comment

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

nit: new line. Same in other files

Copy link
Collaborator

@pmishchenko-ua pmishchenko-ua May 30, 2025

Choose a reason for hiding this comment

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

new: move to line 3 to keep the imports sorted. Same in other files

unique_together={("name", "slug")},
),
]

Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: new line. Same in other files

Book.objects.create(read=70, unread=10)
Book.objects.create(read=70, unread=30)


Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: remove empty line here

@psharma-1909 psharma-1909 merged commit 9f40fd7 into singlestore-test-4.2.x Jun 11, 2025
2 of 3 checks passed
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.

3 participants