Skip to content

ci: trigger PR tests on changes to 'deploy' Kustomize project and related script for deployment#84

Merged
rm3l merged 1 commit intoredhat-developer:mainfrom
rm3l:ci/trigger_pr_tests_on_changes_to_deploy_kustomize_project
Mar 11, 2026
Merged

ci: trigger PR tests on changes to 'deploy' Kustomize project and related script for deployment#84
rm3l merged 1 commit intoredhat-developer:mainfrom
rm3l:ci/trigger_pr_tests_on_changes_to_deploy_kustomize_project

Conversation

@rm3l
Copy link
Member

@rm3l rm3l commented Mar 11, 2026

No description provided.

@rhdh-qodo-merge
Copy link

Review Summary by Qodo

Trigger PR tests on deploy changes

✨ Enhancement

Grey Divider

Walkthroughs

Description
• Trigger PR tests on changes to deploy Kustomize project
• Trigger PR tests on changes to hack/deploy-k8s.sh script
Diagram
flowchart LR
  A["PR Changes"] -->|"deploy/** or hack/deploy-k8s.sh"| B["Trigger test.yaml workflow"]
  B --> C["Run PR Tests"]
Loading

Grey Divider

File Changes

1. .github/workflows/test.yaml ⚙️ Configuration changes +2/-0

Add deploy paths to workflow triggers

• Added deploy/** to workflow trigger paths
• Added hack/deploy-k8s.sh to workflow trigger paths
• Ensures PR tests run when deployment configuration or scripts change

.github/workflows/test.yaml


Grey Divider

Qodo Logo

@rhdh-qodo-merge
Copy link

rhdh-qodo-merge bot commented Mar 11, 2026

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo

@rm3l rm3l merged commit 708f740 into redhat-developer:main Mar 11, 2026
3 checks passed
@rm3l rm3l deleted the ci/trigger_pr_tests_on_changes_to_deploy_kustomize_project branch March 11, 2026 23:30
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