Skip to content

Fix MIN() to handle date comparison correctly - #2479

Open
Aston8 wants to merge 1 commit into
AlaSQL:developfrom
Aston8:patch-1
Open

Aston8 wants to merge 1 commit into
AlaSQL:developfrom
Aston8:patch-1

Conversation

@Aston8

@Aston8 Aston8 commented Apr 19, 2026

Copy link
Copy Markdown

Thank you for the time you are putting into AlaSQL!

@gabrielmongefranco

gabrielmongefranco commented Apr 19, 2026

Copy link
Copy Markdown

Hello, I just noticed this pull request. Please see #2481 and #2480.

@mathiasrw

Copy link
Copy Markdown
Member

Thank you for this.

How can we be sure we did not break something?

@mathiasrw

Copy link
Copy Markdown
Member

Any chance you can add a test file?

@gabrielmongefranco

Copy link
Copy Markdown

Hi, let me dig through the git history in my repo to understand the issue better. I remember I had to do some work arounds but I can't remember why. :)

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