-
Notifications
You must be signed in to change notification settings - Fork 1
MS-1288 Change tracking PR-checks #1531
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR implements smart change tracking for the CI/CD pipeline to optimize test execution by only running tests for modules that have changed. The workflow now detects which module groups have been modified and conditionally executes corresponding test suites.
Changes:
- Added change detection using path filters to identify modified module groups
- Refactored test execution to batch all Gradle tasks and run them in a single command
- Added conditional execution logic to skip test jobs when their modules haven't changed
- Configured concurrency control to cancel in-progress runs when new commits are pushed
Reviewed changes
Copilot reviewed 1 out of 1 changed files in this pull request and generated 11 comments.
| File | Description |
|---|---|
.github/workflows/pr-checks.yml |
Added changes detection job with path filters for different module groups, conditional execution for all test jobs, concurrency control, and updated Sonar scan trigger conditions |
.github/workflows/reusable-run-unit-tests.yml |
Refactored test execution to build a single Gradle command string from all modules instead of running tests in a loop |
Comments suppressed due to low confidence (1)
.github/workflows/pr-checks.yml:16
- There's a grammatical error in the comment: 'Ensures that Sonar check are run' should be 'Ensures that Sonar checks are run'.
# Ensures that Sonar check are run once PR is ready
- ready_for_review
4511b8a to
f0a11fc
Compare
8f2309a to
5db4741
Compare
b6378aa to
f9fa76b
Compare
5db4741 to
c140d48
Compare
f9fa76b to
d786432
Compare
JIRA ticket
Notable changes
Additional work checklist