Re: [PATCH] automation: add missing "needs"

[Stefano Stabellini <sstabellini@xxxxxxxxxx>]

On Fri, 3 Aug 2023, Michal Orzel wrote:
> On 04/08/2023 03:10, Stefano Stabellini wrote:
> > 
> > 
> > From: Stefano Stabellini <stefano.stabellini@xxxxxxx>
> > 
> > The test artifacts export jobs don't need prerequisites, so we should
> > specify "needs" with an empty array for them. That way, they are not
> > going to fruitlessly wait for previous jobs (ECLAIR jobs) to complete
> > before continuing.
> > 
> > Signed-off-by: Stefano Stabellini <stefano.stabellini@xxxxxxx>
> Reviewed-by: Michal Orzel <michal.orzel@xxxxxxx>

Thanks!

> Looking at the whole pipeline, I can see that now only ECLAIR jobs do not 
> specify needs.
> Shouldn't we also add "needs: []" for them?

Yes it could be good for consistency and future-proofness, but
technically it is not required now because the ECLAIR jobs come first.