Skip to content
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

Skip Duplicate GitHub Actions #676

Merged
merged 3 commits into from
Dec 17, 2024

Conversation

lenvm
Copy link
Collaborator

@lenvm lenvm commented Dec 16, 2024

What

This PR updates the GitHub Workflow (a.k.a. GitHub Action) that is triggered to build all batteries.

Why

This change is introduced, because currently 88 checks are performed by GitHub Actions when an additional commit is pushed to an existing pull request, and many of these checks are duplicates. The duplicates arise as the workflows are triggered both by the push as well as by the pull_request trigger. Introducing this change, only 46 checks are performed, therefore significantly reducing the amount of builds done 🌍.

Removing either the push or pull_request trigger was also considered, but does not lead to the desired behavior (see this discussion for reference How to trigger an action on push or pull request but not both? #26276).

How

It does this by introducing the following GitHub Action in the workflow: Skip Duplicate Actions.

@lenvm lenvm changed the title add skip-duplicate-actions to batteries workflow Skip Duplicate GitHub Actions Dec 16, 2024
Copy link
Owner

@dalathegreat dalathegreat left a comment

Choose a reason for hiding this comment

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

Very good improvement! Those 88 checks were becoming a bit heavy yes 😅

@dalathegreat dalathegreat merged commit 9994bae into dalathegreat:main Dec 17, 2024
46 checks passed
@lenvm lenvm deleted the feature/skip-duplicate-actions branch December 20, 2024 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants