Verwijder automatische test-deploy bij PR's
Continuous Integration / config (pull_request) Successful in 9s
Continuous Integration / prepare (pull_request) Successful in 1m18s
Continuous Integration / build-production (pull_request) Skipped
Continuous Integration / build (pull_request) Successful in 2m0s
Continuous Integration / test (pull_request) Successful in 1m53s
Continuous Integration / deploy-test (pull_request) Skipped
Continuous Integration / deploy-production (pull_request) Skipped
Continuous Integration / config (pull_request) Successful in 9s
Continuous Integration / prepare (pull_request) Successful in 1m18s
Continuous Integration / build-production (pull_request) Skipped
Continuous Integration / build (pull_request) Successful in 2m0s
Continuous Integration / test (pull_request) Successful in 1m53s
Continuous Integration / deploy-test (pull_request) Skipped
Continuous Integration / deploy-production (pull_request) Skipped
De tijdelijke pull_request-conditie in deploy-test zorgde dat elke PR automatisch naar test deployde. Dat is niet meer gewenst: een test-deploy vanaf een feature-branch kan al zonder PR via een handmatige workflow_dispatch-run met 'Deploy to Test' aangevinkt. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
@@ -254,16 +254,15 @@ jobs:
|
||||
|
||||
deploy-test:
|
||||
needs: [build, test, config]
|
||||
# TIJDELIJK: 'pull_request' is toegevoegd zodat elke PR ook automatisch
|
||||
# naar de testomgeving deployed, om testen tijdens deze werkbranch te
|
||||
# vereenvoudigen. Verwijder de 'pull_request'-conditie hieronder weer
|
||||
# zodra dat niet meer nodig is.
|
||||
#
|
||||
# De push-naar-master- en pull_request-condities hieronder zijn bewust
|
||||
# ONAFHANKELIJK van het 'deploy_test'-vinkje: de automatische testdeploy
|
||||
# bij een merge naar master moet altijd blijven gebeuren. Het vinkje
|
||||
# geldt alleen voor handmatige workflow_dispatch-runs.
|
||||
if: (github.event_name == 'workflow_dispatch' && github.event.inputs.deploy_test == 'true') || github.event_name == 'pull_request' || (github.event_name == 'push' && github.ref == 'refs/heads/master')
|
||||
# De push-naar-master-conditie hieronder is bewust ONAFHANKELIJK van het
|
||||
# 'deploy_test'-vinkje: de automatische testdeploy bij een merge naar
|
||||
# master moet altijd blijven gebeuren. Het vinkje geldt alleen voor
|
||||
# handmatige workflow_dispatch-runs. Een PR triggert hier bewust GEEN
|
||||
# deploy: wil je een feature-branch naar test deployen, start dan zelf
|
||||
# een workflow_dispatch-run op die branch met 'Deploy to Test' aangevinkt
|
||||
# (Gitea: Actions → Continuous Integration → Run workflow → branch
|
||||
# selecteren) - dat vereist geen (afgeronde) PR.
|
||||
if: (github.event_name == 'workflow_dispatch' && github.event.inputs.deploy_test == 'true') || (github.event_name == 'push' && github.ref == 'refs/heads/master')
|
||||
uses: ./.gitea/workflows/deploy.yaml
|
||||
secrets: inherit
|
||||
# Deze waarden komen uit het `env:`-blok bovenaan dit bestand, via de
|
||||
|
||||
Reference in New Issue
Block a user