Skip to content

Conversation

MauricioFauth
Copy link
Member

Moves the StatementInfo class from the phpMyAdmin's source code into the sql-parser's code.

It replaces the string value of the querytype field of the statement
flags.

Signed-off-by: Maurício Meneghini Fauth <mauricio@fauth.dev>
Signed-off-by: Maurício Meneghini Fauth <mauricio@fauth.dev>
Signed-off-by: Maurício Meneghini Fauth <mauricio@fauth.dev>
Copy link

codecov bot commented Feb 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (8857557) 96.54% compared to head (2de0fdc) 96.54%.
Report is 3 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #550      +/-   ##
============================================
- Coverage     96.54%   96.54%   -0.01%     
  Complexity     2149     2149              
============================================
  Files            86       87       +1     
  Lines          5006     5004       -2     
============================================
- Hits           4833     4831       -2     
  Misses          173      173              

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

Copy link
Contributor

@kamil-tekiela kamil-tekiela left a comment

Choose a reason for hiding this comment

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

I love it! Great job!

@MauricioFauth MauricioFauth merged commit c4efadf into phpmyadmin:master Feb 5, 2024
@MauricioFauth MauricioFauth deleted the stmt-info branch February 5, 2024 17:02
@MauricioFauth MauricioFauth self-assigned this Feb 5, 2024
@MauricioFauth MauricioFauth added this to the 6.0.0 milestone Feb 5, 2024
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