Skip to content

chore(deps): hydra-gates 1.10, so the E2E skip-discipline gate can run - #943

Merged
rubenvdlinde merged 1 commit into
developmentfrom
chore/hydra-gates-1.10
Aug 28, 2026
Merged

chore(deps): hydra-gates 1.10, so the E2E skip-discipline gate can run#943
rubenvdlinde merged 1 commit into
developmentfrom
chore/hydra-gates-1.10

Conversation

@rubenvdlinde

Copy link
Copy Markdown
Contributor

Why

conduction/hydra-gates v1.10.0 contains check_e2e_skips.py, which the shared quality workflow needs to run the E2E skip-discipline check. On v1.9.0 that script does not exist, so the step reports

::warning::check_e2e_skips.py not found (hydra-gates not vendored here yet)
— skip discipline NOT measured on this run.

and the job goes green having measured nothing. Today that is the state of 20 of 21 fleet apps — only launchpad, which was bumped in ConductionNL/launchpad#359, produces a verdict at all.

A skip is counted as a pass, and the Playwright report is the only place a runtime test.skip(cond, reason) records its reason — the CI log prints counts without reasons. So until this lands, an app cannot tell a legitimate skip from a fixture that silently never ran. Context in ConductionNL/.github#609.

Lock-only

composer.json already allows this — the constraint is ^1.8.2 and 1.10.0 satisfies it. Only composer.lock moves.

Blast radius, measured before opening this

v1.10.0 adds six gate scripts over v1.9.0:

check_adr_number_collision   check_manifest_copy_style   check_retired_git_host
check_e2e_skips              check_repair_registration   check_system_elevation

So this is not a no-op bump, and it is deliberately opened as a PR rather than pushed: the gates run here and report before anything can redden development.

gate-99 manifest-l10n-coverage was run locally against every fleet app first, because that is the gate that caught launchpad when it was bumped. 18 of 20 are clean; this app is one of them. The exception is zaakafhandelapp (59 manifest strings with no nl.json key), which needs its l10n gap closed before it can take this bump.

@rubenvdlinde
rubenvdlinde merged commit ba401eb into development Aug 28, 2026
44 checks passed
@rubenvdlinde
rubenvdlinde deleted the chore/hydra-gates-1.10 branch August 28, 2026 08:27
@github-actions

Copy link
Copy Markdown
Contributor

Quality Report — ConductionNL/decidiq @ 4944275

Check PHP Vue Security License Tests
lint
phpcs
phpmd
psalm
phpstan
phpmetrics
eslint
stylelint
build
check-manifest
check-nav-ceiling
test-l10n
format
check-l10n-js
check-schema-l10n
composer ✅ 104/104
npm ✅ 557/557
app:check-code ⏭️
info.xml
REUSE
PHPUnit
Newman
Playwright
Hydra gates

Quality workflow — 2026-08-28 08:30 UTC

Download the full PDF report from the workflow artifacts.

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