Skip to content

feat: capture env vars as properties file during action run - #83

Merged
marcphilipp merged 6 commits into
mainfrom
marc/env-properties-file
Aug 6, 2026
Merged

feat: capture env vars as properties file during action run#83
marcphilipp merged 6 commits into
mainfrom
marc/env-properties-file

Conversation

@marcphilipp

Copy link
Copy Markdown
Member

To avoid problems with running tests in Docker containers or VMs, the
environment variables that are inherent to thee GitHub Actions
environment (GITHUB_*, RUNNER_*) and the custom JOB_CHECK_RUN_ID
are now captured by the action. The file is then read by the
instrumentation during test execution rather than relying on the env
vars being propagated.

@marcphilipp
marcphilipp force-pushed the marc/env-properties-file branch from 68f8ab0 to c8ee08c Compare August 5, 2026 13:37
@marcphilipp
marcphilipp marked this pull request as ready for review August 5, 2026 13:38
@marcphilipp
marcphilipp force-pushed the marc/env-properties-file branch from c8ee08c to a8093ab Compare August 5, 2026 14:58
To avoid problems with running tests in Docker containers or VMs, the
environment variables that are inherent to thee GitHub Actions
environment (`GITHUB_*`, `RUNNER_*`) and the custom `JOB_CHECK_RUN_ID`
are now captured by the action. The file is then read by the
instrumentation during test execution rather than relying on the env
vars being propagated.
Rather than relying on the `CI` env var to be present, the injected
profile is now active by default. This avoids having to propagate the
`CI` env var to Docker containers or VMs that execute the Maven build.
Otherwise, Docker containers and VMs won't see it.
Comment thread setup-testlens.sh Outdated
Base automatically changed from bats-tests to main August 6, 2026 09:14
@marcphilipp
marcphilipp merged commit 636fd3b into main Aug 6, 2026
7 checks passed
@marcphilipp
marcphilipp deleted the marc/env-properties-file branch August 6, 2026 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants