Skip to content

Add SQLite to test matrix#54

Open
Copilot wants to merge 6 commits intomainfrom
copilot/add-sqlite-to-test-matrix
Open

Add SQLite to test matrix#54
Copilot wants to merge 6 commits intomainfrom
copilot/add-sqlite-to-test-matrix

Conversation

Copy link

Copilot AI commented Jan 19, 2026

Adds SQLite testing to the automated test matrix to provide visibility into SQLite compatibility across WP-CLI packages.

Changes

  • Added dbtype matrix dimension (default: mysql)
  • Added SQLite test entries for 6 packages (wp-cli-bundle, wp-cli, cache-command, config-command, core-command, entity-command) on PHP 8.5 + WP trunk
  • Added pdo_sqlite PHP extension and WP_CLI_TEST_DBTYPE environment variable
  • Skip MySQL setup and database preparation when dbtype == 'sqlite'
  • SQLite tests marked with continue-on-error: true to prevent CI pipeline blocking
  • Updated job names to display database type

Example test configuration:

- php: '8.5'
  wp: 'trunk'
  dbtype: 'sqlite'
  use-phar: false
  test-package: 'core-command'
Original prompt

This section details on the original issue you should resolve

<issue_title>Add SQLite to test matrix</issue_title>
<issue_description>All individual packages run tests against SQLite already and for most of them they already pass.

Adding SQLite to the test matrix in this repo would make it easier to see SQLite compatibility at a glance.</issue_description>

<agent_instructions>SQLite tests should be allowed to fail</agent_instructions>

Comments on the Issue (you are @copilot in this section)

@danielbachhuber Seems fine 👍

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

Copilot AI and others added 2 commits January 19, 2026 10:27
Co-authored-by: swissspidy <841956+swissspidy@users.noreply.github.com>
Co-authored-by: swissspidy <841956+swissspidy@users.noreply.github.com>
Copilot AI changed the title [WIP] Add SQLite to test matrix for compatibility checks Add SQLite to test matrix Jan 19, 2026
Copilot AI requested a review from swissspidy January 19, 2026 10:31
@swissspidy swissspidy marked this pull request as ready for review January 19, 2026 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add SQLite to test matrix

2 participants

Comments