Skip to content

ci: validate the reduced on-code-change inputs (do not merge) - #726

Closed
baptistegrimaud wants to merge 2 commits into
mainfrom
ci/validate-reduced-on-code-change-inputs
Closed

ci: validate the reduced on-code-change inputs (do not merge)#726
baptistegrimaud wants to merge 2 commits into
mainfrom
ci/validate-reduced-on-code-change-inputs

Conversation

@baptistegrimaud

@baptistegrimaud baptistegrimaud commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Do not merge — throw-away

This pull request exists only to run Jahia/jahia-modules-action#405 against a real repository. It pins reusable-on-code-change.yml to that branch and drops the seven inputs the branch is supposed to derive. It will be deleted once the run has been read.

Result

Run 32479908553 is green, integration tests included, and every dropped input resolved as intended. The evidence table lives in jahia-modules-action#405. This pull request can be deleted.

What it proves

The caller keeps four inputs instead of eleven:

with:
  module_id: javascript-modules-engine
  build_container_image: ghcr.io/jahia/jahia-docker-mvn-cache:17-jdk-noble-mvn-loaded
  integration_tests_provisioning_manifest: provisioning-manifest-build.yml
  integration_tests_testrail_project: Javascript Modules Engine
Dropped input What the run must show
module_branch every job checks out this pull request's ref
sonar_analysis_primary_release_branch the Sonar analysis compares against main, resolved from the base of this pull request
integration_tests_standalone_execute the standalone integration tests run, inferred from the provisioning manifest
integration_tests_jahia_image the tests start ghcr.io/jahia/jahia-ee-dev:8-SNAPSHOT
integration_tests_should_skip_testrail TestRail stays skipped
integration_tests_should_use_build_artifacts the build artifacts are still downloaded
static_analysis_auditci_level audit-ci still runs at critical

build_container_image stays because this repository builds on JDK 17, not the shared default of 11.

Throw-away: pins the reusable workflow to the branch of
Jahia/jahia-modules-action#405 and drops the inputs it derives.
@github-actions

github-actions Bot commented Aug 21, 2026

Copy link
Copy Markdown

🦜 Chachalog

No changelog entries detected. Learn more about Chachalog.

Create a new entry online or run npx chachalog@0.5.4 prompt to create a new entry locally.

@pkg-pr-new

pkg-pr-new Bot commented Aug 21, 2026

Copy link
Copy Markdown

Open in StackBlitz

yarn add https://pkg.pr.new/@jahia/create-module@726.tgz
yarn add https://pkg.pr.new/@jahia/javascript-modules-library@726.tgz
yarn add https://pkg.pr.new/@jahia/vite-plugin@726.tgz

commit: fc4cb01

@baptistegrimaud

Copy link
Copy Markdown
Contributor Author

Superseded: jahia-modules-action#405 is merged and released in v2.65.0, so the reduced caller now lands properly in #732 against @v2. This throw-away pull request has served its purpose.

@baptistegrimaud
baptistegrimaud deleted the ci/validate-reduced-on-code-change-inputs branch August 21, 2026 15:48
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.

1 participant