-
-
Notifications
You must be signed in to change notification settings - Fork 111
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
chore(linux): Fix bug with incomplete parameter in run-tests.sh, add dependency and update documentation #9602
Conversation
User Test ResultsTest specification and instructions User tests are not required Test Artifacts
|
c5e9436
to
704ba55
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Could you come up with a more descriptive title please? 😁 |
When running the integration tests, we provide a test service that replaces `keyman-system-service`. DBus needs the updated `.service` files in the `services` directory so that it will use the test service. This change outputs an error and returns an non-zero exit code if it can't find the `services` directory, which usually means that `keyman-system-service` didn't get build.
ad01caf
to
5d074b0
Compare
For integration tests ibus-keyman requires keyman-system-service to be built, so we add this as a dependency.
5d074b0
to
3d8cf18
Compare
This change updates gha-ubuntu-packaging to a version that creates a non-empty changelog entry. This prevents a warning during package build.
This no longer suppresses the output of cleanup.
Changes in this pull request will be available for download in Keyman version 17.0.178-alpha |
Hmm, 'fix bugs' is still not clear! Would be better to state which bugs this PR is fixing |
services
directory@keymanapp-test-bot skip