Skip to content

fix: avoid use of builder in tests.inc.sh for CI - #105

Merged
mcdurdin merged 1 commit into
mainfrom
fix/tests-no_builder
Sep 15, 2026
Merged

mcdurdin merged 1 commit into
mainfrom
fix/tests-no_builder

Conversation

@mcdurdin

Copy link
Copy Markdown
Member

CI does not use builder when calling tests. So do not use builder functions.

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 B19S2 milestone Sep 15, 2026
@github-actions github-actions Bot added the fix label Sep 15, 2026
@github-project-automation github-project-automation Bot moved this to Todo in Keyman Sep 15, 2026
CI does not use builder when calling tests. So do not use builder
functions. Also fixup Windows vs non-Windows docker calls to handle
root path correctly.

Test-bot: skip
@mcdurdin
mcdurdin force-pushed the fix/tests-no_builder branch from 262a86a to caf6f13 Compare September 15, 2026 06:45

@rc-swag rc-swag left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

lgtm

@mcdurdin
mcdurdin merged commit 50306bf into main Sep 15, 2026
3 checks passed
@mcdurdin
mcdurdin deleted the fix/tests-no_builder branch September 15, 2026 07:15
@github-project-automation github-project-automation Bot moved this from Todo to Done in Keyman Sep 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants