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

Change the order of tasks to run #13258

Closed
de-don opened this issue Nov 18, 2022 · 1 comment · Fixed by #13294
Closed

Change the order of tasks to run #13258

de-don opened this issue Nov 18, 2022 · 1 comment · Fixed by #13294
Labels

Comments

@de-don
Copy link

de-don commented Nov 18, 2022

Description

Sort the projects to run starting from the bottom of the graph and ending at the top, in order to first run the task on low-level libraries, and at the end - on high-level ones

Motivation

I noticed that if you make changes to a low-level library (on which many libraries depend) and run affected:test, then the modified library is not tested first, but about in the middle of the queue.
It may be optimal to start from the lowest level of the graph (from the library that has changed) and move up.

Let's say I have a library in my project that 100 other libraries depend on. Having made changes to it, I expect that it will be tested first, and if the test falls, I will interrupt the rest of the tests and go to fix the error.

At the moment, I didn't find in the source code that before starting a task, projects are somehow sorted, and because of this, the launch is almost in random order:

function projectsToRun(

image

Suggested Implementation

As I have already described above, it is logical to perform tasks first at the lower levels of the tree, and move up.

@github-actions
Copy link

This issue has been closed for more than 30 days. If this issue is still occuring, please open a new issue with more recent context.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants