Skip to content

Replace the cancel previous run by the concurrency #324

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

Draft
wants to merge 12 commits into
base: develop
Choose a base branch
from

Conversation

byhbt
Copy link
Contributor

@byhbt byhbt commented Mar 28, 2023

What happened 👀

  • Replace the step Cancel Previous Runs in all the workflow. Replace it by the concurrency group
  • Add new team member to the CODEOWNER

Insight 📝

As the suggestion from the creator of Cancel Previous Runs:

image

Reference: https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#concurrency

Proof Of Work 📹

image

@byhbt byhbt self-assigned this Mar 28, 2023
@@ -15,17 +15,16 @@ env:
HEROKU_API_KEY: ${{ secrets.HEROKU_API_KEY }}
HEROKU_APP_NAME: ${{ secrets.HEROKU_APP_NAME }}

concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: ${{ github.ref != 'refs/heads/main' }}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shall we cancel it on the develop branch as well 🤔 because we deploy to Staging from the develop

@byhbt byhbt force-pushed the chore/remove-cancel-previous-run branch from 4ccb1bb to 6b15723 Compare April 8, 2023 06:53
Copy link
Contributor

@olivierobert olivierobert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@byhbt let's complete this PR as there is not much left to do, no?

@byhbt
Copy link
Contributor Author

byhbt commented Aug 1, 2023

@byhbt let's complete this PR as there is not much left to do, no?

yup, i was checking about the Andy suggestion 😅 , let me update it and make it ready for review today or tomorrow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants