No changes needed - code already uses count() > 0 per feedback

[Copilot]

Addressed review feedback on PR #276 regarding the use of empty() vs count() > 0 for array emptiness checks.

Context

A bot suggested changing count( $unkwnown_columns ) > 0 to ! empty( $unkwnown_columns ) for readability. @JanJakes objected, noting that empty() is problematic and potentially slower.

Resolution

The code already uses count( $unkwnown_columns ) > 0 in commit 8bd98ae. The bot's suggestion was never implemented. No code changes required.

// Current implementation (unchanged)
$unkwnown_columns = array_diff( $insert_list, array_column( $columns, 'COLUMN_NAME' ) );
if ( count( $unkwnown_columns ) > 0 ) {
    throw $this->new_driver_exception(/* ... */);
}

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[JanJakes]

Oh, Copilot 🤦