Skip to content

Conversation

@Tigrov
Copy link
Member

@Tigrov Tigrov commented Nov 17, 2024

Rellated PR

Q A
Is bugfix?
New feature?
Breaks BC?
Fixed issues

@codecov
Copy link

codecov bot commented Nov 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.50%. Comparing base (cc707f8) to head (511e5c1).

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #327      +/-   ##
============================================
- Coverage     98.50%   98.50%   -0.01%     
  Complexity      336      336              
============================================
  Files            23       23              
  Lines          1073     1072       -1     
============================================
- Hits           1057     1056       -1     
  Misses           16       16              

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

protected const AUTO_INCREMENT_KEYWORD = 'AUTOINCREMENT';

protected const GENERATE_UUID_EXPRESSION =
"(unhex(format('%016X', random() & 0xFFFFFFFFFFFF4FFF | 0x4000) || format('%016X', random() & 0xBFFFFFFFFFFFFFFF | 0xB000000000000000)))";
Copy link
Member

Choose a reason for hiding this comment

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

Is that v4 or v7?

Copy link
Member Author

@Tigrov Tigrov Nov 17, 2024

Choose a reason for hiding this comment

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

Valid UUID v4.
Also there is randomblob(16) function but it is not valid UUID.

@Tigrov Tigrov merged commit 51919cb into master Nov 17, 2024
@Tigrov Tigrov deleted the refactor-column-definition-builder branch November 17, 2024 13:16
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