Skip to content

Conversation

@stloyd
Copy link
Member

@stloyd stloyd commented Oct 20, 2023

Change Log

Added

Fixed

Changed

  • Reduce CI load by cancelling previous runs with every new commit in PR

Removed

Deprecated

Security


Description

Based on: https://stackoverflow.com/questions/66335225/how-to-cancel-previous-runs-in-the-pr-when-you-push-new-commitsupdate-the-curre/72408109#72408109

@norberttech norberttech merged commit d4ba15c into flow-php:1.x Oct 20, 2023
@stloyd stloyd deleted the chore/ci-concurrency branch October 20, 2023 18:56
@norberttech
Copy link
Member

hmm I'm getting job cancellation notifications, isn't that conflicting with post merge jobs also?

@stloyd
Copy link
Member Author

stloyd commented Oct 20, 2023

Maybe just turn off those cause they are not useful as cancellation should happen on manual trigger or particular failure of infrastructure.

@norberttech
Copy link
Member

oh I know what's happening, whenver we merge anything, automation is also pushing changes to the CHANGELOG.md file, which triggers all tests all over again. Could you take a look into this and try to figure out if there is any way to prevent triggering the whole testsuite after the update of CHANGELOG.md?

@stloyd
Copy link
Member Author

stloyd commented Oct 22, 2023

@norberttech Should be fixed with: #623

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.

2 participants