Commit graph

2000 commits

Author SHA1 Message Date
Marc Durdin
f8c63ed65d
Merge pull request #14893 from keymanapp/maint/trigger-more-logging
maint(resources): fix watch bug for common files, and add more logging for triggering builds
2025-10-10 05:03:31 +02:00
Marc Durdin
69922d0ba3 maint(resources): further logging and fix indirection bug with watch 2025-10-06 16:55:02 +02:00
Eberhard Beilharz
65f2e354eb
maint(linux): suppress output of stopping processes after test run
This change suppresses the output of ending the processes started for
the test run. The output doesn't help much but clutters the overall
output and shows up as warnings on TC.
2025-10-06 16:13:44 +02:00
Eberhard Beilharz
229bdf8270
maint(resources): nest unit tests under parent block
This adds a modified `mocha-teamcity-reporter` that allows to nest the
unit tests under the parent block.

Fixes: #14839
Test-bot: skip
2025-10-06 16:09:47 +02:00
Marc Durdin
ebc137ab11 maint(resources): more logging for triggered builds
Add a list of all files modified in the PR; removes 'web' from windows
build target.

Test-bot: skip
2025-10-06 15:44:38 +02:00
Eberhard Beilharz
6c2237d800
Merge pull request #14817 from keymanapp/maint/linux/13887_integrtests
maint(linux): fix integration tests in Docker

This PR allows the integration tests to work when running them in a Docker container. Also fixes things when running on TC's Ubuntu 24.04 Noble build agent, and improves the behavior when running directly on a developer's machine (previously typing stopped working after the tests...).

Also fixes building mcompile in a Docker container.

Fixes: #13887
2025-10-06 12:01:38 +02:00
Eberhard Beilharz
b302da5094
refactor(web): address code review comments 2025-10-02 10:47:49 +02:00
Eberhard Beilharz
34c43b1e15
refactor(web): cleanup in test scripts
Also use common function for running web types tests.

Test-bot: skip
2025-09-30 17:40:15 +02:00
Markus Greiner
49d8acf16b running the container for core in privileged mode
this is expected to fix a permission issue when running in
GitHub Actions.

Test-bot: skip
Build-bot: skip
2025-09-30 12:57:16 +02:00
Eberhard Beilharz
167ec33d54
chore(linux): Merge remote-tracking branch 'origin/master' into maint/linux/remotedbgdocker 2025-09-30 08:29:02 +02:00
Eberhard Beilharz
cca4a502f1
docs(linux): document --remote-debug option for run.sh 2025-09-29 17:49:00 +02:00
Eberhard Beilharz
16727304b5
chore(linux): fix building mcompile in Docker container 2025-09-25 19:34:22 +02:00
Marc Durdin
df194a7511
Merge pull request #14816 from keymanapp/maint/resources/find-7z-on-path-in-windows
maint(resources): lookup 7z on path on Windows if SEVENZ_HOME is not set
2025-09-25 15:41:48 +02:00
Marc Durdin
bf9e135072 maint(resources): use command instead of where to find 7z.exe
Using `7z.exe` instead of `7z` finds the correct command.
2025-09-25 08:56:06 +02:00
Marc Durdin
ec229c1a23
Merge pull request #14815 from keymanapp/maint/windows/sign-check-only-in-ci
maint(windows): check code signatures only in CI builds
2025-09-25 05:48:02 +02:00
Markus Greiner
b8007beedc fix(common): registry clean-up enabled, no terminal in github action
No longer dryrun for the registry clean-up GHA workflow
docker run doesn't use a terminl (erasing -t flag)

Test-bot: skip
Build-bot: skip
2025-09-24 17:24:16 +02:00
Eberhard Beilharz
db3cd5092d
maint(linux): remove several shellcheck directives and warnings
This change improves our `.shellcheckrc` file so that several directives
are no longer needed and source files are still found. Also remove some
directives that are no longer necessary, either because the changes to
`.shellcheckrc` makes them obsolete, or because the lines changed so
that they no longer contain the problem.

Also remove some shellcheck warnings.

Build-bot: skip
Test-bot: skip
2025-09-23 18:30:16 +02:00
Eberhard Beilharz
23f8c59c07
maint(linux): allow to remote debug integration tests in Docker
This adds a `--remote-debug` option to `resources/docker-images/run.sh`.
If specified the container will expose `localhost:2345` for remote
debugging inside of the container.

This is useful for debugging integration tests in the container.

Build-bot: skip
Test-bot: skip
2025-09-23 18:20:32 +02:00
Marc Durdin
4828df7c89 maint(resources): lookup 7z on path on Windows if SEVENZ_HOME is not set
Also cleans up a number of possible quoting issues and shellcheck
directives.

Note: we need to do release builds everywhere to verify that zip changes
are not going to break things.

Test-bot: skip
Build-bot: release
2025-09-23 10:44:58 +02:00
Marc Durdin
c06b5b8eea maint(windows): check code signatures only in CI builds
When building locally, code signatures will not have SIL in the name, so
will fail the build. So skip checking code signatures on local builds.

Build-bot: skip release:windows,developer
Test-bot: skip
2025-09-23 10:32:55 +02:00
Markus Greiner
ea77b0cb5f Merge remote-tracking branch 'origin/master' into fix/common/tests-running-in-container
adding "-t" when Docker containers are not started by a Docker container
2025-09-22 14:04:26 +02:00
Eberhard Beilharz
14d47cc14d
refactor(resources): use array for Docker arguments
Test-bot: skip
2025-09-19 18:45:31 +02:00
Markus Greiner
bae371bb50 fix(common): some more tests running successfully in a container
setting KEYMAN_ROOT
ignoring test:linux until #13887 is fixed
removing docker "-t" option, because there is no terminal to attach
while running on the GitHub server

Test-bot: skip
Build-bot: skip
2025-09-18 11:24:53 +02:00
Eberhard Beilharz
f628a869ea
maint(resources): fix broken test build trigger for master branch
Test-bot: skip
2025-09-17 11:24:27 +02:00
Marc Durdin
d852916d4d
Merge pull request #14734 from keymanapp/fix/resources/pr-build-status-cr-to-lf
fix(resources): match cr to lf change in pr-build-status.mjs
2025-09-12 09:48:07 +02:00
Darcy Wong
3a0e031be4
Merge pull request #14728 from keymanapp/chore/common/unicode-17
chore(common): Update to Unicode 17.0
2025-09-12 09:05:50 +07:00
Marc Durdin
293dbb83e9 fix(resources): match cr to lf change in pr-build-status.mjs
Follows: #14732
Build-bot: skip
Test-bot: skip
2025-09-11 14:59:51 +02:00
Marc Durdin
6d6cc36c8f fix(resources): add more logging for build-status 2025-09-11 13:25:45 +02:00
Marc Durdin
7e3925ebfc fix(resources): always pass summary through
Fixes: #14731
2025-09-11 13:18:43 +02:00
Marc Durdin
1433f15506 fix(resources): pr build status - find 'failure' state in status checks and log
Fixes: #14731
Build-bot: skip
Test-bot: skip
2025-09-11 12:59:16 +02:00
Marc Durdin
ebeacc83d0
Merge branch 'master' into chore/common/unicode-17 2025-09-11 11:53:03 +02:00
Marc Durdin
a49b0146ce fix(resources): ignore KEYMAN_VERSION_FOR_FILENAME in test
Follows: #14703
Test-bot: skip
Build-bot: skip build:common
2025-09-11 06:49:27 +02:00
Darcy Wong
8321294e1e chore(common): Update to Unicode 17.0 2025-09-11 11:28:47 +07:00
Marc Durdin
d9622ff4b1
chore(developer): whitespace in builder-basic.inc.sh
Build-bot: skip

Co-authored-by: Eberhard Beilharz <ermshiperete@users.noreply.github.com>
2025-09-10 15:07:09 +02:00
Marc Durdin
377b1056ec maint(resources): rename KEYMAN_VERSION_FOR_PR_FILENAME
Address review comment from #14703.

Test-bot: skip
Build-bot: skip
Relates-to: #14703
Relates-to: #10521
2025-09-10 11:17:41 +02:00
Marc Durdin
8a2adf5161
Merge pull request #14704 from keymanapp/maint/ios/10521-use-versioned-filenames-for-pr-artifacts
maint(ios): use versioned filenames for PR artifacts
2025-09-10 11:15:14 +02:00
Marc Durdin
50f86cfbf7
Merge pull request #14703 from keymanapp/maint/android/10521-use-versioned-filenames-for-pr-artifacts
maint(android): use versioned filenames for PR artifacts
2025-09-10 11:15:02 +02:00
Markus Greiner
1cf644a0b8
Merge pull request #14674 from keymanapp/maint/common/13954-create-gha-for-docker-images
maint(resources): adding GitHub actions workflow for Docker builds
2025-09-10 10:01:36 +02:00
Markus Greiner
535f7aa153 maint(resources): added trigger for Docker images build, pubish, test
moving example script to readme file
adding delete job for container images in ghcr

Part-of: #13594
Test-bot: skip
Build-bot: skip
2025-09-08 17:28:11 +02:00
Eberhard Beilharz
ab33d638f5
maint(linux): check that command is at beginning of line
Addresses code review comments.

Co-Authored-By: Marc Durdin <marc@durdin.net>
2025-09-08 16:52:56 +02:00
Markus Greiner
5a4635c883 Update resources/docker-images/README.md
Co-authored-by: Eberhard Beilharz <ermshiperete@users.noreply.github.com>
2025-09-08 15:26:44 +02:00
Markus Greiner
f8f6626f55 main(resources): #13954 adding GitHub actions workflow for Docker builds
Test-bot: skip
Build-bot: skip
2025-09-08 15:26:44 +02:00
Marc Durdin
18ebb0baf3 maint(ios): use versioned filename for testflight upload
Build-bot: skip build:ios
2025-09-08 13:29:03 +02:00
Marc Durdin
634fef64d4 maint(android): use versioned filenames for PR artifacts
Relates-to: #10521
Test-bot: skip
Build-bot: skip release:android,developer,windows
2025-09-08 12:31:11 +02:00
Eberhard Beilharz
612cd01743
maint(linux): allow to skip API check
This change adds an option to skip the API checks during a build.
This can be done by adding the trailer `Keyman-Api-Check: skip`
to the PR description.

Fixes: #10454
Test-bot: skip
Keyman-Api-Check: skip
2025-09-05 19:04:23 +02:00
Eberhard Beilharz
a9597b58da
refactor(resources): remove shellcheck warnings 2025-09-05 19:02:16 +02:00
Eberhard Beilharz
f809d6a77f
Merge pull request #14695 from keymanapp/maint/resources/comments
Git allows to specify a comment character different from the default `#`. This is helpful when using markdown in commit messages, especially when adding a user test section that has to start with `# User testing` in order for the test bot to detect it.

This change modifies the `prepare-commit-msg` hook and adds a check for a configured `core.commentString` or `core.commentChar`. If both are not set it falls back to the default `#` character.

You can set a different comment character with `git config core.commentChar //`.
2025-09-05 15:36:37 +02:00
Eberhard Beilharz
1becb87547
maint(resources): honor configure comment character in commit message
Git allows to specify a comment character different from the default
`#`. This is helpful when using markdown in commit messages, especially
when adding a user test section that has to start with `# User testing`
in order for the test bot to detect it.

This change modifies the `prepare-commit-msg` hook and adds a check for
a configured `core.commentString` or `core.commentChar`. If both are not
set it falls back to the default `#` character.

You can set a different comment character with
`git config core.commentChar //`.

Build-bot: skip
Test-bot: skip
2025-09-05 10:45:58 +02:00
Eberhard Beilharz
5c9e4f2c08
maint(linux): remove obsolete Makefile
All commands from `linux/Makefile` are now available elsewhere and not
used anymore. The one exception was `make tmpsources` - this change
moves the code to the TC build script.

Test-bot: skip
2025-09-05 10:40:16 +02:00
Eberhard Beilharz
e615139348
Merge pull request #14669 from keymanapp/chore/windows/desktop 2025-09-03 11:53:46 +02:00