Skip to content

chore: tweak test controls in Docker - #110

Merged
mcdurdin merged 1 commit into
mainfrom
chore/docker-test-controls
Oct 29, 2026
Merged

mcdurdin merged 1 commit into
mainfrom
chore/docker-test-controls

Conversation

@mcdurdin

Copy link
Copy Markdown
Member

Allow build.sh to set DOCKER_NO_xx test variables to disable certain types of tests. And add a log reporting function to centralize finding PHP errors after tests.

Test-bot: skip

@keymanapp-test-bot

Copy link
Copy Markdown

User Test Results

Test specification and instructions

User tests are not required

@keymanapp-test-bot keymanapp-test-bot Bot added this to the B19S5 milestone Oct 29, 2026
@github-actions github-actions Bot added the chore label Oct 29, 2026
Allow build.sh to set DOCKER_NO_xx test variables to disable certain
types of tests. And add a log reporting function to centralize finding
PHP errors after tests.

Test-bot: skip
@mcdurdin
mcdurdin force-pushed the chore/docker-test-controls branch from 9cc6a07 to 7a56adc Compare October 29, 2026 08:20
Comment thread _common/docker.inc.sh
# tag in the shared-sites repo. We will do this with a heuristic based on the
# normal shape of the tags, which is `v#.#.#`. Any other shape we will assume is
# a test branch.
function docker_verify_bootstrap_required_version() {

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.

This has been copied straight out of ci.yml from various sites as we start to centralize these tests. (Also docker_report_php_errors)

@mcdurdin
mcdurdin merged commit 5efcf3f into main Oct 29, 2026
3 checks passed
@mcdurdin
mcdurdin deleted the chore/docker-test-controls branch October 29, 2026 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants