aleph

dergutemoritz 2025-09-23T08:38:14.387809Z

Reposting here for greater visibility.

dergutemoritz 2025-10-06T07:40:57.470169Z

Great, thank you! Will get back to it today

dergutemoritz 2025-10-02T08:21:12.918719Z

Thanks! Can you resolve your comments and approve the respective PRs for posterity? 🙏 Not sure if you need extra permission for the latter though

dergutemoritz 2025-10-20T09:05:17.330079Z

Indeed, it's big and also quite subtle. I hope the PR description helps 🙏

👍 1
valerauko 2025-10-04T02:34:52.577959Z

I can't resolve comments but approved

dergutemoritz 2025-10-01T09:46:11.337249Z

Thanks a lot for your feedback so far! Already responded to some of it but gotta run now. Will get back to the remaining one tomorrow! 🙏

valerauko 2025-10-01T21:43:32.555809Z

looks good!

dergutemoritz 2025-10-11T15:46:08.979289Z

Got interrupted by a cold 😬 Back to you 🙂 🙏

dergutemoritz 2025-10-22T14:23:04.038089Z

Thanks a lot for your feedback once more! Back to you 🙂

👍 1
valerauko 2025-09-29T13:58:49.172609Z

Other than what I commented on it lgtm 🚀

🙏 1
dergutemoritz 2025-09-29T14:05:58.892689Z

Great, thanks 🙂 Do you also feel up to reviewing the follow-up patches?

dergutemoritz 2025-09-29T14:06:09.653159Z

Or did you already?

valerauko 2025-09-29T14:17:02.474859Z

I didn't see those. I'll take a look tomorrow

dergutemoritz 2025-09-29T14:20:14.023279Z

That's great, thank you!

dergutemoritz 2025-09-29T14:20:22.533279Z

They're linked from the main one

dergutemoritz 2025-10-16T08:54:51.285229Z

Thanks a lot! Now there's only one remaining which indeed makes the tests pass again 🙂 It's https://github.com/clj-commons/manifold/pull/252. I asked the original author of the code in question to review it but you're still welcome to check it out, as well, of course 🙂

valerauko 2025-10-16T14:23:47.758869Z

that's a big one huh. i'll see if i can find some time tomorrow to look at it