Naitik Shah

createRelease order race condition

1

There is a race (I just hit this) where `createRelease` fails because between trying to get it and create it, another build in the matrix already created it. A retry...

Consider renaming `useContainer` to just `use`?

1

Neat library to remove some redundant code. Just a thought, about naming. Feel free to close as you see fit.

I traced back the history to 92a2263ad39c61f6812a870a79fe4fe49576899c but it's unclear why `:handler` is removed from the `match-context` (along with other info which unlike this is present in the request). I...

The readme says the `ego` CLI recursively finds templates and processes them. This isn't true - it only goes 1 level deep. This change makes it recursive just like the...

There's a note saying it's doesn't show in usage in the code.. but why not?

`StartMonitorEvents` takes a callback, and a channel, and untyped args. It spins up a new goroutine per call, but shares an atomic to allow `StopAllMonitorEvents` to work. The whole thing...

A less dom and more template approach

4

Currently the way the `rsx!` macro parses and outputs something akin to a psuedo-dom style set of element and attributes is similar to what React does. But the current use...

Use io::Write instead of fmt::Write

1

From the docs of [`fmt::Write`](https://doc.rust-lang.org/std/fmt/trait.Write.html): > This trait is the type which this modules requires when formatting information. This is similar to the standard library's io::Write trait, but it is...

Instant cannot be cast to Duration

3

I was trying to upgrade from `0.11.3` to `0.11.6` and one of my tests failed with this. Still investigating but possibly related to e82bf79560f1365f50a38f5e8f40714a3f6be53b? ``` #error { :cause "class java.time.Instant...

ignore element and children in morph

2

I'm using this successfully for an improved experience with various input fields in forms (via htmx). Part of my replace target though includes large number of elements, which I know...