Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(common): add unit test for --debug flag for builder #10974

Merged
merged 1 commit into from
Mar 16, 2024

Conversation

mcdurdin
Copy link
Member

@mcdurdin mcdurdin commented Mar 11, 2024

Fixes #10957. The builder --debug flag appears to be passed correctly, according to this test. This is a good test to keep, so won't remove it, but there is something else going on in the base #10953 PR.

@keymanapp-test-bot skip

Fixes #10957. The builder --debug flag appears to be passed correctly,
according to this test. This is a good test to keep, so won't remove it,
but there is something else going on in the base test.
@mcdurdin mcdurdin requested a review from jahorton as a code owner March 11, 2024 10:47
@keymanapp-test-bot keymanapp-test-bot bot added this to the B17S3 milestone Mar 11, 2024
@mcdurdin mcdurdin linked an issue Mar 11, 2024 that may be closed by this pull request
Copy link
Contributor

@darcywong00 darcywong00 left a comment

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 b101460 into beta Mar 16, 2024
26 checks passed
@mcdurdin mcdurdin deleted the fix/common/10957-builder-debug-dep-flag branch March 16, 2024 02:25
@keyman-server
Copy link
Collaborator

Changes in this pull request will be available for download in Keyman version 17.0.289-beta

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.

bug(common): builder is not appearing to pass debug flags correctly
3 participants