Skip to content

Testing - IR integration enable concurrency#536

Merged
dpasukhi merged 10 commits intoOpen-Cascade-SAS:IRfrom
dpasukhi:ir_use_no_cancel_2
May 15, 2025
Merged

Testing - IR integration enable concurrency#536
dpasukhi merged 10 commits intoOpen-Cascade-SAS:IRfrom
dpasukhi:ir_use_no_cancel_2

Conversation

@dpasukhi
Copy link
Copy Markdown
Member

Fix concurrency cancel-in-progress condition in CI workflow for IR branch

Fix concurrency cancel-in-progress condition in CI workflow for IR branch
@dpasukhi dpasukhi requested a review from Copilot May 15, 2025 14:37
@dpasukhi dpasukhi self-assigned this May 15, 2025
@dpasukhi dpasukhi added the 1. Testing Updating of the DRAW/GTests or GH tests label May 15, 2025
Copy link
Copy Markdown

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 addresses an issue with the concurrency cancel-in-progress condition in the CI workflow for the IR branch.

  • Updated the concurrency group expression to distinguish pull request events from other run types.
  • Revised the cancel-in-progress expression to use github.head_ref for pull request events.
Comments suppressed due to low confidence (1)

.github/workflows/build-and-test-multiplatform.yml:23

  • Verify that using github.head_ref for branch comparison in pull request events (instead of github.ref) is the intended behavior, ensuring it correctly identifies the IR branch.
&& !(github.head_ref == 'IR' && github.repository == 'Open-Cascade-SAS/OCCT')

Comment thread .github/workflows/build-and-test-multiplatform.yml Outdated
@dpasukhi dpasukhi requested a review from Copilot May 15, 2025 14:50
Copy link
Copy Markdown

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 addresses a concurrency cancel-in-progress issue in the CI workflow for the IR branch. The changes refactor the concurrency group and cancel-in-progress logic to account for pull request events versus other events.

  • Updated concurrency group expression to differentiate between pull request and non-pull request events.
  • Modified the cancel-in-progress condition to target the IR branch more accurately.

@dpasukhi dpasukhi requested a review from AtheneNoctuaPt May 15, 2025 14:51
@dpasukhi dpasukhi added this to the Release 8.0 milestone May 15, 2025
@github-project-automation github-project-automation bot moved this from Todo to Integration in Maintenance May 15, 2025
@dpasukhi dpasukhi merged commit ebc3885 into Open-Cascade-SAS:IR May 15, 2025
60 of 64 checks passed
@dpasukhi dpasukhi deleted the ir_use_no_cancel_2 branch May 15, 2025 18:34
@github-project-automation github-project-automation bot moved this from Integration to Done in Maintenance May 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1. Testing Updating of the DRAW/GTests or GH tests

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants