CodesInChaos

Another problematic feature is: `#[cfg_attr(feature = "with-serde", derive(serde::Serialize, serde::Deserialize))]` on the existing struct. This leaks the format of the private fields into json. I would suggest adding manual implementations of...

If we use `PartialOrd` to limit `BoundedRange` to non-empty ranges, then making `Range` an enum works. That seems like the cleanest design, even if means that constructing a `BoundedRange` from...

Unfortunately there were some unexpected challenges: **NaN handling** Edgedb treats `NaN` as larger than any normal value and equal to itself, while rust treats it as incomparable. I solved this...

If there is a negative zero (i.e. `negative = true` but all digits being zero), then a `wire_negative` function should be added instead, and `is_negative` should return `false` for negative...

Another option would be to sign in the user if it's on the same device, and requiring them to sign in again if it's a different device. This would be...

> Why not use rustfmt for everything? Rustfmt adds/removes optional linebreaks based on line length (being "pretty printer"). I'm not fond of that feature. Mat describes a less aggressive formatter...

Why should this be limited to computed properties? Including or excluding a property from `select {*}` seems orthogonal to computed vs physical properties.

Duplicate of #3540 "Allow mutating simple computeds"

I think the ability to [extend tuples](https://github.com/geldata/gel/issues/7650) would cover similar use-cases, but add the ability to define constraints, making it more useful (though probably harder to implement as well).

I'm experiencing the same crash. As a workaround, disabling "Online features" seems to avoid the problem.