Skip to content

fix: fix rewrite to traverse Sel in ast.In #2414

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jul 12, 2023
Merged

fix: fix rewrite to traverse Sel in ast.In #2414

merged 1 commit into from
Jul 12, 2023

Conversation

@orisano
Copy link
Contributor

@orisano orisano commented Jul 7, 2023

fix #2403
@orisano orisano changed the title fix: fix to traverse Sel in ast.In Jul 7, 2023
@orisano orisano changed the title fix: fix rewrite to traverse Sel in ast.In Jul 11, 2023
@orisano orisano changed the title fix(engine/sqlite): fix rewrite to traverse Sel in ast.In Jul 11, 2023
@andrewmbenton
Copy link
Collaborator

Thanks! I am going to merge this and then create an endtoend test using the input from the playground link provided in #2403: https://play.sqlc.dev/p/4db6630bb57b14ff1913c150a2c0140e9eb4e587f54e93586f430e99681ae83b
@andrewmbenton andrewmbenton merged commit 156bd6d into sqlc-dev:main Jul 12, 2023
@andrewmbenton andrewmbenton mentioned this pull request Jul 12, 2023
Merged
alfonsodev pushed a commit to ExponentiaTeam/sqlc that referenced this pull request Oct 13, 2025
@orisano
fix: fix to traverse Sel in ast.In (sqlc-dev#2414)
b4727b8 
fix sqlc-dev#2403
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@orisano @andrewmbenton