Skip to content

Remove USE_UTF_STRINGS#18491

Merged
MauricioFauth merged 1 commit intophpmyadmin:masterfrom
kamil-tekiela:USE_UTF_STRINGS
Jun 9, 2023
Merged

Remove USE_UTF_STRINGS#18491
MauricioFauth merged 1 commit intophpmyadmin:masterfrom
kamil-tekiela:USE_UTF_STRINGS

Conversation

@kamil-tekiela
Copy link
Copy Markdown
Contributor

phpmyadmin/sql-parser#471

Signed-off-by: Kamil Tekiela <tekiela246@gmail.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 7, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.02 ⚠️

Comparison is base (70ff7dc) 55.95% compared to head (c6ddb1e) 55.94%.

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #18491      +/-   ##
============================================
- Coverage     55.95%   55.94%   -0.02%     
+ Complexity    16083    16080       -3     
============================================
  Files           635      635              
  Lines         63458    63458              
============================================
- Hits          35510    35503       -7     
- Misses        27948    27955       +7     
Flag Coverage Δ
dbase-extension 55.94% <100.00%> (+<0.01%) ⬆️
recode-extension 55.91% <100.00%> (+<0.01%) ⬆️
unit-8.1-ubuntu-latest ?
unit-8.2-ubuntu-latest ?
unit-nightly-ubuntu-latest ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
libraries/classes/Linter.php 100.00% <100.00%> (+1.88%) ⬆️

... and 1 file with indirect coverage changes

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

@MauricioFauth MauricioFauth merged commit fa8931e into phpmyadmin:master Jun 9, 2023
@MauricioFauth MauricioFauth self-assigned this Jun 9, 2023
@kamil-tekiela kamil-tekiela deleted the USE_UTF_STRINGS branch June 9, 2023 22:50
@williamdes williamdes added this to the 6.0.0 milestone Jun 10, 2023
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