Skip to content

Conversation

@luhmirin-s
Copy link
Contributor

JIRA ticket

Notable changes

  • Sonar is only run if PR is ready for review
  • When new commit is pushed to a PR, previous run will be cancelled
  • PR-check will not be triggered if only documentation or GH files have changed
  • All test tasks in group are run with a single gradle command to cut overhead

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)

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 optimizes GitHub Actions workflows to reduce CI execution time and resource usage. It implements several performance improvements including conditional Sonar scanning, workflow run cancellation on new commits, smarter path filtering, and batched Gradle task execution.

Changes:

  • Sonar scans now only run when PRs are ready for review or triggered manually
  • New commits to PRs automatically cancel in-progress workflow runs
  • Workflows skip when only documentation or GitHub configuration files change
  • Unit tests now execute all modules in a single Gradle command instead of sequential individual commands

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
.github/workflows/pr-checks.yml Adds path filtering for docs/GitHub files, concurrency control for canceling in-progress runs, and conditional execution of Sonar scans
.github/workflows/reusable-run-unit-tests.yml Refactors test execution to aggregate all module tasks into a single Gradle command with --continue flag

@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-pr-check-optimisation branch from 5db4741 to c140d48 Compare January 13, 2026 16:48
@sonarqubecloud
Copy link

@luhmirin-s luhmirin-s merged commit 06e852b into main Jan 14, 2026
13 checks passed
@luhmirin-s luhmirin-s deleted the feature/MS-1288-pr-check-optimisation 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