Skip to content

Add DISTINCT keyword support to Query classes#27

Merged
byjg merged 2 commits into
masterfrom
5.0
May 22, 2025
Merged

Add DISTINCT keyword support to Query classes#27
byjg merged 2 commits into
masterfrom
5.0

Conversation

@byjg

@byjg byjg commented May 22, 2025

Copy link
Copy Markdown
Owner

Implemented support for the DISTINCT keyword in both Query and QueryBasic classes, enabling distinct queries. Added tests to confirm correctness and ensure compatibility across the classes.

Description by Korbit AI

What change is being made?

Add support for the DISTINCT keyword in the Query and QueryBasic classes, including corresponding unit tests.

Why are these changes being made?

This change enhances the querying capabilities by allowing the DISTINCT keyword, addressing the need to filter out duplicate results in query outputs. Implementing tests ensures the new feature works as intended and does not introduce regressions into the existing codebase.

Is this description stale? Ask me to generate a new description by commenting /korbit-generate-pr-description

Implemented support for the DISTINCT keyword in both Query and QueryBasic classes, allowing for distinct queries. Added corresponding tests to validate this functionality and ensure compatibility across both classes.

@korbit-ai korbit-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review by Korbit AI

Korbit automatically attempts to detect when you fix issues in new commits.
Category Issue Status
Functionality Undefined 'distinct' Property ▹ view
Files scanned
File Path Reviewed
src/Query.php
src/QueryBasic.php

Explore our documentation to understand the languages and file types we support and the files we ignore.

Check out our docs on how you can make Korbit work best for you and your team.

Loving Korbit!? Share us on LinkedIn Reddit and X

Comment thread src/Query.php
@byjg byjg merged commit 7e7414c into master May 22, 2025
4 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.

1 participant