janpeter/Ops-dashboard
1
0
Fork 0
Code Issues Pull requests 1 Projects Releases Packages Wiki Activity Actions

fix(deploy): trigger logt Engels, in lijn met de rest van de scripts #139

Merged
janpeter merged 1 commit from fix/trigger-log-language into main 2026-08-02 18:58:39 +02:00
Conversation 1 Commits 1 Files changed 2 +12 -12
janpeter commented 2026-08-02 18:48:20 +02:00
Owner
Copy link

De laatste Nederlandse operator-output in dit subsysteem. Dit script draait bij elke merge, dus het is ook de regel die het vaakst gelezen wordt.

Zelfde onderbouwing als bij de SKIPPED-melding: setup.sh, control-room-foundation.sh, update-operator-mcp.sh en install-network-module.sh loggen allemaal Engels, en de foutcodes ook.

Tien logregels vertaald. Alleen de log-output — de comments blijven zoals ze zijn, er is geen bredere sweep gevraagd.

De tests moesten mee

Twee assertions hingen aan de Nederlandse strings:

toContain('OPGEGEVEN')   -> toContain('GIVING UP')
toContain('poging 1/3')  -> toContain('attempt 1/3')

Zonder die aanpassing waren ze rood geworden op een puur cosmetische wijziging. 7/7 groen erna.

Echt gedraaid

[scrum4us-deploy-trigger] NOOP: checkout is up to date (9f91631e87d9) — nothing to deploy
exit 0
De laatste Nederlandse operator-output in dit subsysteem. Dit script draait bij **elke merge**, dus het is ook de regel die het vaakst gelezen wordt. Zelfde onderbouwing als bij de SKIPPED-melding: `setup.sh`, `control-room-foundation.sh`, `update-operator-mcp.sh` en `install-network-module.sh` loggen allemaal Engels, en de foutcodes ook. Tien logregels vertaald. **Alleen de log-output** — de comments blijven zoals ze zijn, er is geen bredere sweep gevraagd. ## De tests moesten mee Twee assertions hingen aan de Nederlandse strings: ``` toContain('OPGEGEVEN') -> toContain('GIVING UP') toContain('poging 1/3') -> toContain('attempt 1/3') ``` Zonder die aanpassing waren ze rood geworden op een puur cosmetische wijziging. 7/7 groen erna. ## Echt gedraaid ``` [scrum4us-deploy-trigger] NOOP: checkout is up to date (9f91631e87d9) — nothing to deploy exit 0 ```
fix(deploy): trigger logt Engels, in lijn met de rest van de scripts
All checks were successful
CI / Root app checks (pull_request) Successful in 5m55s
Details
CI / Ops-agent checks (pull_request) Successful in 15s
Details
CI / Deploy artifact checks (pull_request) Successful in 12s
Details
CI / Docker image build (pull_request) Successful in 1m23s
Details
3a122523cc 
De laatste Nederlandse operator-output in dit subsysteem. Dit script draait bij elke
merge, dus het is ook de regel die het vaakst gelezen wordt. Zelfde reden als bij de
SKIPPED-melding: setup.sh, control-room-foundation.sh, update-operator-mcp.sh en
install-network-module.sh loggen allemaal Engels, en de foutcodes ook.

Tien logregels vertaald. Alleen de log-output; de comments blijven zoals ze zijn, want
er is geen bredere sweep gevraagd.

De tests asserteerden op twee van die strings ('OPGEGEVEN', 'poging 1/3') en zijn
meegegaan naar 'GIVING UP' en 'attempt 1/3'. Zonder dat waren ze stilzwijgend rood
geworden op een puur cosmetische wijziging.

Echt gedraaid na installatie:
  [scrum4us-deploy-trigger] NOOP: checkout is up to date (9f91631e87d9) — nothing to deploy
  exit 0

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
s4m-codex-reviewer approved these changes 2026-08-02 18:51:06 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: APPROVED

Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

Findings

  • Geen blokkerende of error-severity findings.

Review-notities

De wijziging beperkt zich tot het vertalen van runtime-logregels in deploy/ops-agent/scripts/scrum4us-deploy-trigger.sh en het bijwerken van de bijbehorende testverwachtingen in test/scrum4us-deploy-trigger.test.ts. Control-flow, exitcodes, lockgedrag, fail-counter-state en deploy-aanroep blijven ongewijzigd. De geraadpleegde productdocs bevatten geen strijdige conventie voor deze logtekst.

# Verdict: APPROVED Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - Geen blokkerende of error-severity findings. ## Review-notities De wijziging beperkt zich tot het vertalen van runtime-logregels in `deploy/ops-agent/scripts/scrum4us-deploy-trigger.sh` en het bijwerken van de bijbehorende testverwachtingen in `test/scrum4us-deploy-trigger.test.ts`. Control-flow, exitcodes, lockgedrag, fail-counter-state en deploy-aanroep blijven ongewijzigd. De geraadpleegde productdocs bevatten geen strijdige conventie voor deze logtekst.
Sign in to join this conversation.
Reviewers
No reviewers
s4m-codex-reviewer
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
janpeter/Ops-dashboard!139
No description provided.