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

Refactor build method #483

Merged
merged 2 commits into from
Aug 20, 2023

Conversation

kamil-tekiela
Copy link
Contributor

This overrides the previous two PRs.
This isn't even 50% of what I wanted to achieve but fixing this is really difficult. As I am going to be off of the internet for the next month, I will not have time to work on this.

@codecov
Copy link

codecov bot commented Jun 11, 2023

Codecov Report

Patch coverage: 93.33% and project coverage change: -0.16% ⚠️

Comparison is base (d70e65d) 96.61% compared to head (2930a89) 96.46%.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #483      +/-   ##
============================================
- Coverage     96.61%   96.46%   -0.16%     
- Complexity     2173     2177       +4     
============================================
  Files            66       66              
  Lines          5054     5062       +8     
============================================
  Hits           4883     4883              
- Misses          171      179       +8     
Files Changed Coverage Δ
src/Components/Array2d.php 90.47% <0.00%> (-4.77%) ⬇️
src/Components/ExpressionArray.php 91.66% <50.00%> (-3.99%) ⬇️
src/Components/UnionKeyword.php 45.45% <50.00%> (-10.11%) ⬇️
src/Components/JoinKeyword.php 94.11% <66.66%> (-2.86%) ⬇️
src/Components/AlterOperation.php 98.67% <100.00%> (ø)
src/Components/ArrayObj.php 100.00% <100.00%> (ø)
src/Components/CaseExpression.php 98.47% <100.00%> (ø)
src/Components/Condition.php 100.00% <100.00%> (ø)
src/Components/CreateDefinition.php 100.00% <100.00%> (ø)
src/Components/Expression.php 100.00% <100.00%> (ø)
... and 19 more

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

@williamdes williamdes marked this pull request as draft June 17, 2023 18:33
@williamdes
Copy link
Member

I think it needs a rebase, I marked it as draft

@kamil-tekiela kamil-tekiela force-pushed the Refactor-build-method branch 2 times, most recently from 44e5c60 to ddb7a3f Compare July 27, 2023 10:50
@kamil-tekiela kamil-tekiela marked this pull request as ready for review July 27, 2023 10:55
Signed-off-by: Kamil Tekiela <[email protected]>
Signed-off-by: Kamil Tekiela <[email protected]>
@MauricioFauth MauricioFauth merged commit 52ffc49 into phpmyadmin:master Aug 20, 2023
10 of 12 checks passed
@MauricioFauth MauricioFauth self-assigned this Aug 20, 2023
@kamil-tekiela kamil-tekiela deleted the Refactor-build-method branch August 20, 2023 20:27
@williamdes williamdes added this to the 6.0.0 milestone Aug 21, 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