-
-
Notifications
You must be signed in to change notification settings - Fork 244
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
Doubling CI jobs issue for internal contributors #2377
Comments
@nvuillam the culprit is me in #2294 but we really need a way to run the build+tests of each linter in parallel. Until now it was only done with a very small subset fixed in the workflows and for me in that PR (#2294) it was super frustrating to have to wait for the task that builds all linters and runs all the tests together (Build MegaLinter Docker Image (full from main repo) and Run Test Cases) and wait half an hour or more for it to fail or not.... By running them in parallel at least it speeds up that process quite a bit.... |
I know that, but the is no reason to run twice the same test for a single PR :) |
I stumbled upon this, when I wasn't looking specifically for this https://github.com/fkirc/skip-duplicate-actions. I didn't make my opinion on it yet. But maybe we could try temporarily setting |
This issue has been automatically marked as stale because it has not had recent activity. If you think this issue should stay open, please remove the |
I think I solved it, now the linter jobs are run just once as they are triggered only on pull_request, not push :) |
We have 234 CI jobs, it's really too many and when there are several running PR workflows, it generates too much delay
I didn't succeed to configure Build Deploy DEV - Linters job to avoid it to be launched twice... if someone succeeds it it would really improve contributing experience :)
Another solution could be to add a condition to run individual linter checks, so they would be triggered only when we want it, for example if the commit text or PR text contains some word like
check linters
The text was updated successfully, but these errors were encountered: