[11:21] <librasteve_> lizmat: are you ok for me to merge https://github.com/croservices/cro-http/pull/214 ?

[11:23] <librasteve_> (I just want to have a 2nd pair of eyes for deep changes like this in case any possible side-effects I have not considered - then I can go ahead with a point release of cro-http)

[11:46] <Geth> ¦ cro-http/main: a2949b6443 | (Fritz Zaucker)++ (committed using GitHub Web editor) | lib/Cro/HTTP/Middleware.rakumod

[11:46] <Geth> ¦ cro-http/main: Fix unbounded memory growth in Conditional middleware (#214)

[11:46] <Geth> ¦ cro-http/main: 

[11:46] <Geth> ¦ cro-http/main: The Conditional middleware's Response transform taps the per-connection

[11:46] <Geth> ¦ cro-http/main: `early-responses` Supplier but never completes it. The RequestResponse

[11:46] <Geth> ¦ cro-http/main: middleware completes the same Supplier via `LAST ... .done` when its

[11:46] <Geth> ¦ cro-http/main: response pipeline ends; Conditional omitted this, so the Supplier (and the

[11:46] <Geth> ¦ cro-http/main: connection-scoped state reachable from its subscription) was retained,

[11:46] <Geth> ¦ cro-http/main: <…commit message has 11 more lines…>

[11:46] <Geth> ¦ cro-http/main: review: https://github.com/croservices/cro-http/commit/a2949b6443

[12:10] <lizmat> librasteve_   yes :-)

[12:45] <disbot4> <librasteve> ah - already done - tx

[12:48] <librasteve_> croservices CI failure https://www.irccloud.com/pastebin/MoRpOkVv

[12:49] <librasteve_> hi - we got ^^ this failure https://github.com/croservices/cro-openapi-routes-from-definition/actions/runs/25521057363/job/78649520851 on the PR checks on croservices — any raku CI folks who know the fix?

[12:51] <lizmat> raku-version: 2021.03  ???

[12:51] <lizmat> feels like a YAML fix is needed >

[12:51] <lizmat> ?

[12:54] <lizmat> librasteve_: removed the reference to 2021.03...  testing only latest Raky

[12:54] <lizmat> Raku

[12:55] <lizmat> not sure why Geth didn't report

[12:57] <disbot4> <librasteve> what is the platform test strategy for Cro - do we / should we span (or cover) "an old one" + "latest" [I had naively thought that we would want to not drop the quite old 2021.03 since that has been working all along]

[12:58] <disbot4> <librasteve> I am happy to drop it, but would just like to understand the plan

[12:59] <lizmat> well, the test failed because the image for 2021.03 could not be found

[13:11] <lizmat> fwiw, I've added an organization-wide webhook, so all croservices repos will report here

[13:11] <lizmat> and removed any other webhooks from the cro repos (they were either identical or non-functional)

[13:13] <Geth> ¦ cro-openapi-routes-from-definition/main: f970092dba | (Elizabeth Mattijsen)++ (committed using GitHub Web editor) | .github/workflows/ci.yml

[13:13] <Geth> ¦ cro-openapi-routes-from-definition/main: Add Raku version 2022.04 to CI workflow

[13:13] <Geth> ¦ cro-openapi-routes-from-definition/main: review: https://github.com/croservices/cro-openapi-routes-from-definition/commit/f970092dba

[13:28] <disbot4> <librasteve> ok, capiche

[13:47] <Geth> ¦ cro-openapi-routes-from-definition/main: a38c4db639 | (Fritz Zaucker)++ (committed using GitHub Web editor) | 2 files

[13:47] <Geth> ¦ cro-openapi-routes-from-definition/main: Fix OperationHandler for Cro::HTTP::Router 0.8.12+

[13:47] <Geth> ¦ cro-openapi-routes-from-definition/main: 

[13:47] <Geth> ¦ cro-openapi-routes-from-definition/main: Cro::HTTP 0.8.12 added a per-handler `.name` lookup and `.url-prefix`

[13:47] <Geth> ¦ cro-openapi-routes-from-definition/main: assignment in `Cro::HTTP::Router::RouteSet.definition-complete` (used

[13:47] <Geth> ¦ cro-openapi-routes-from-definition/main: when a `route` block does `include 'foo' => ...` with a handler from

[13:47] <Geth> ¦ cro-openapi-routes-from-definition/main: this module). The built-in `RouteHandler` declares both attributes;

[13:47] <Geth> ¦ cro-openapi-routes-from-definition/main: `OperationHandler` did not, so any pattern that includes an OpenAPI

[13:47] <Geth> ¦ cro-openapi-routes-from-definition/main: <…commit message has 18 more lines…>

[13:47] <Geth> ¦ cro-openapi-routes-from-definition/main: review: https://github.com/croservices/cro-openapi-routes-from-definition/commit/a38c4db639

[15:00] *** japhb left
[15:09] *** japhb joined
[17:26] *** librasteve_ left
[19:06] *** japhb left
[19:17] *** japhb joined