Skip to content

Conversation

@kamil-tekiela
Copy link
Contributor

Follow up from #451 but this time I move the class to SelectStatement where it belongs due to feature envy.

@codecov
Copy link

codecov bot commented May 31, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (aeb52db) 96.64% compared to head (e47aef6) 96.64%.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #454   +/-   ##
=========================================
  Coverage     96.64%   96.64%           
+ Complexity     2214     2213    -1     
=========================================
  Files            67       66    -1     
  Lines          5096     5096           
=========================================
  Hits           4925     4925           
  Misses          171      171           
Impacted Files Coverage Δ
src/Components/IndexHint.php 100.00% <ø> (ø)
src/Statements/ReplaceStatement.php 100.00% <ø> (ø)
src/Context.php 97.67% <100.00%> (ø)
src/Exceptions/ParserException.php 100.00% <100.00%> (ø)
src/Parser.php 100.00% <100.00%> (ø)
src/Statement.php 100.00% <100.00%> (ø)
src/Statements/SelectStatement.php 100.00% <100.00%> (ø)
src/TokensList.php 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Member

@williamdes williamdes left a comment

Choose a reason for hiding this comment

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

Very nice idea !

@williamdes williamdes added this to the 6.0.0 milestone Jun 1, 2023
Signed-off-by: Kamil Tekiela <[email protected]>
williamdes added a commit that referenced this pull request Jun 1, 2023
Pull-request: #454

Signed-off-by: William Desportes <[email protected]>
@williamdes williamdes merged commit 0d4d75e into phpmyadmin:master Jun 1, 2023
@williamdes williamdes self-assigned this Jun 1, 2023
@kamil-tekiela kamil-tekiela deleted the Misc-2 branch June 7, 2023 18:25
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