Re: [PATCH 1/1] ci: avoid unnecessary builds

From: Taylor Blau <me@ttaylorr.com>
Date: Nov 8, 2022, 20:09 UTC
Message-ID: <Y2q3f5noQSXmK0o8@nand.local>
In-Reply-To: <221108.86r0ydqmts.gmgdl@evledraar.gmail.com>

On Tue, Nov 08, 2022 at 07:52:07PM +0100, Ævar Arnfjörð Bjarmason wrote:

> I.e. the "cancel-in-progress" seems to work perfectly. I pushed a branch
> with this, it started CI, then I made a trivial update, re-pushed, and
> the initial branch's CI run was stopped, one job at a time (not all at
> the same time, but almost).

Thanks to Johannes for all of that ;-).

Show 8 quoted lines

> Now, obviously it's a bit more of a verbose solution v.s. Johannes's
> original in [1], but on the other hand our main.yml is full of that sort
> of copy/pasting already.
>
> Aside from the change under discussion I don't find that to be a big
> deal. It's a general "problem", and we could always generate the file
> with a script if we really cared, but I think for the forseeable future
> the copy/pasting is fine.

To be fair, I think that the solution I came up with is pretty gross. But after spending a few hours with the Actions docs, I think that it's the least-gross solution that meets the needs of our respective workflows.

So I'm OK with it.

Thanks, Taylor