Skip to content

Resolve "Combine all rake based lints in one rake process in CI"

What does this MR do?

This simply runs all the Rake lint tasks in a single call instead of separate calls.

The previous total duration of all the Rake lint calls was approximately 185 seconds (https://gitlab.com/gitlab-org/gitlab-ce/-/jobs/49720426), with this change it's now 70 seconds (https://gitlab.com/gitlab-org/gitlab-ce/-/jobs/49720426), a 62% improvement.

Are there points in the code the reviewer needs to double check?

Now that all the tasks are run in a single call, the offense report could be less pretty...

Why was this MR needed?

To speed-up the CI job.

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes #42050 (closed)

Edited by Rémy Coutable

Merge request reports