Skip to content

Fix column definition parsing in cases with brackets and escaped quotes#1109

Merged
vjik merged 2 commits into
masterfrom
fix-column-definition-parser
Nov 27, 2025
Merged

Fix column definition parsing in cases with brackets and escaped quotes#1109
vjik merged 2 commits into
masterfrom
fix-column-definition-parser

Conversation

@vjik

@vjik vjik commented Nov 27, 2025

Copy link
Copy Markdown
Member

@codecov

codecov Bot commented Nov 27, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.62%. Comparing base (b005f4d) to head (8de4a3c).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #1109       +/-   ##
=============================================
+ Coverage     84.58%   98.62%   +14.03%     
  Complexity     1648     1648               
=============================================
  Files           119      119               
  Lines          4268     4283       +15     
=============================================
+ Hits           3610     4224      +614     
+ Misses          658       59      -599     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@vjik vjik requested a review from a team November 27, 2025 07:00
@vjik vjik added the status:code review The pull request needs review. label Nov 27, 2025
$matches[1],
);

return ['enumValues' => $values];

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
return ['enumValues' => $values];
return ['values' => $values];

@vjik vjik Nov 27, 2025

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I'll change it in a separate PR

@vjik vjik merged commit b78617d into master Nov 27, 2025
84 of 107 checks passed
@vjik vjik deleted the fix-column-definition-parser branch November 27, 2025 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status:code review The pull request needs review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants