Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Remove Misc class#454

Merged
williamdes merged 2 commits intophpmyadmin:masterphpmyadmin/sql-parser:masterfrom
kamil-tekiela:Misc-2kamil-tekiela/sql-parser:Misc-2Copy head branch name to clipboard
Jun 1, 2023
Merged

Remove Misc class#454
williamdes merged 2 commits intophpmyadmin:masterphpmyadmin/sql-parser:masterfrom
kamil-tekiela:Misc-2kamil-tekiela/sql-parser:Misc-2Copy head branch name to clipboard

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.

src/Statements/SelectStatement.php Show resolved Hide resolved
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 <tekiela246@gmail.com>
Signed-off-by: Kamil Tekiela <tekiela246@gmail.com>
williamdes added a commit that referenced this pull request Jun 1, 2023
Pull-request: #454

Signed-off-by: William Desportes <williamdes@wdes.fr>
@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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.