Skip to content
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.

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

Make parens optional for * EXCEPT and minor clean up #106

Merged
merged 3 commits into from
Apr 30, 2024
Merged

Conversation

KingMob
Copy link
Contributor

@KingMob KingMob commented Apr 19, 2024

This makes parentheses optional for the SELECT * EXCEPT and GENERATE * EXCEPT code, which were not originally created that way.

It also adds a few missing parse tests, and simplifies and aligns their parse trees and plans.

Copy link

codecov bot commented Apr 19, 2024

Codecov Report

Attention: Patch coverage is 80.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 75.94%. Comparing base (8041a2f) to head (4af32f6).

Files Patch % Lines
src/inferenceql/query/plan.cljc 75.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #106      +/-   ##
==========================================
+ Coverage   75.91%   75.94%   +0.03%     
==========================================
  Files          30       30              
  Lines        1536     1534       -2     
  Branches       66       64       -2     
==========================================
- Hits         1166     1165       -1     
- Misses        304      305       +1     
+ Partials       66       64       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@KingMob KingMob force-pushed the push-wlxrpwulplux branch from 875ea6c to bcd9ab4 Compare April 26, 2024 10:20
@KingMob KingMob force-pushed the push-rmqnwwwmmlps branch from f8bff4b to 6ebfca9 Compare April 26, 2024 10:20
@KingMob KingMob force-pushed the push-wlxrpwulplux branch from bcd9ab4 to debd205 Compare April 26, 2024 10:25
@KingMob KingMob force-pushed the push-rmqnwwwmmlps branch from 6ebfca9 to 0110d7e Compare April 26, 2024 10:25
@KingMob KingMob force-pushed the push-wlxrpwulplux branch from debd205 to 3247712 Compare April 26, 2024 15:20
@KingMob KingMob force-pushed the push-rmqnwwwmmlps branch 2 times, most recently from 52a0fc3 to 1529fde Compare April 29, 2024 15:54
@KingMob KingMob force-pushed the push-wlxrpwulplux branch 2 times, most recently from 2cc075b to d14f8ce Compare April 29, 2024 16:33
@KingMob KingMob force-pushed the push-rmqnwwwmmlps branch from 1529fde to 62930fc Compare April 29, 2024 16:33
Base automatically changed from push-rmqnwwwmmlps to main April 30, 2024 09:28
@KingMob KingMob force-pushed the push-wlxrpwulplux branch from d14f8ce to 4af32f6 Compare April 30, 2024 10:16
@KingMob KingMob merged commit 4d2f25c into main Apr 30, 2024
7 checks passed
@KingMob KingMob deleted the push-wlxrpwulplux branch April 30, 2024 14:40
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