Skip to content

Conversation

@luhmirin-s
Copy link
Contributor

@luhmirin-s luhmirin-s commented Jan 13, 2026

JIRA ticket

Notable changes

  • Check the if there were code changes in the modules and only run test group if necessary.
  • Skip tests where nothing has changed.

Additional work checklist

  • Effect on other features and security has been considered
  • Design document marked as "In development" (if applicable)
  • External (Gitbook) and internal (Confluence) Documentation is up to date (or ticket created)
  • Test cases in Testiny are up to date (or ticket created)
  • Other teams notified about the changes (if applicable)

@luhmirin-s luhmirin-s requested review from a team, BurningAXE, TristramN, alex-vt, alexandr-simprints, Copilot, meladRaouf and ybourgery and removed request for a team January 13, 2026 16:29
@cla-bot cla-bot bot added the ... label Jan 13, 2026
@luhmirin-s luhmirin-s changed the base branch from main to feature/MS-1288-pr-check-optimisation January 13, 2026 16:29
Copy link

Copilot AI left a 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

@luhmirin-s luhmirin-s force-pushed the feature/MS-1288-change-tracking-pr-checks branch from 4511b8a to f0a11fc Compare January 13, 2026 16:38
@luhmirin-s luhmirin-s force-pushed the feature/MS-1288-pr-check-optimisation branch from 8f2309a to 5db4741 Compare January 13, 2026 16:38
@luhmirin-s luhmirin-s force-pushed the feature/MS-1288-change-tracking-pr-checks branch 3 times, most recently from b6378aa to f9fa76b Compare January 13, 2026 16:45
@luhmirin-s luhmirin-s force-pushed the feature/MS-1288-pr-check-optimisation branch from 5db4741 to c140d48 Compare January 13, 2026 16:48
@luhmirin-s luhmirin-s force-pushed the feature/MS-1288-change-tracking-pr-checks branch from f9fa76b to d786432 Compare January 13, 2026 16:49
Base automatically changed from feature/MS-1288-pr-check-optimisation to main January 14, 2026 11:40
@luhmirin-s luhmirin-s merged commit 1549b6e into main Jan 14, 2026
12 checks passed
@luhmirin-s luhmirin-s deleted the feature/MS-1288-change-tracking-pr-checks branch January 14, 2026 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants