Skip to content

Conversation

@alexislefebvre
Copy link

@alexislefebvre alexislefebvre commented Nov 29, 2025

Q A
Type improvement
Fixed issues -

Summary

Suggested here:

@alexislefebvre alexislefebvre force-pushed the chore-show-parameters-in-name-of-CI-jobs branch 3 times, most recently from fd498e5 to b4a8b24 Compare November 29, 2025 12:56
SenseException
SenseException previously approved these changes Nov 30, 2025
name: >
PHPUnit with SQLite -
${{ format('OS {0} - PHP {1} - ext. {2} - deps {3}',
matrix.os || 'Ø',
Copy link
Member

Choose a reason for hiding this comment

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

The history might explain why we have os in this matrix, but it's probably not something we will want to display here

Copy link
Member

Choose a reason for hiding this comment

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

Context: PHP uses the libsqlite3 from Ubuntu's package manager. We test with different Ubuntu versions to make sure we also support older SQLite releases. It's the best we can do without compiling SQLite ourselves. The information on the OS is relevant imho. But maybe it's not the most important piece, so we might move it to the back.

@alexislefebvre alexislefebvre force-pushed the chore-show-parameters-in-name-of-CI-jobs branch 2 times, most recently from bc0eec7 to 6e0fce4 Compare November 30, 2025 14:43
@alexislefebvre alexislefebvre force-pushed the chore-show-parameters-in-name-of-CI-jobs branch from 6e0fce4 to cdf646b Compare November 30, 2025 14:50
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.

4 participants