Ross MacLeod

patchThatChangesAndSortsMapWith creates patches with Just undefined as target keys

1

this seems pretty obviously wrong. ImmediateDomBuilderT doesn't actually look at the target key, but it looks like spider would probably blow up. uses of `Just undefined`: https://github.com/reflex-frp/reflex/blob/develop/src/Reflex/Patch/MapWithMove.hs#L129 https://github.com/reflex-frp/reflex/blob/develop/src/Reflex/Patch/MapWithMove.hs#L134 https://github.com/reflex-frp/reflex/blob/develop/src/Reflex/Patch/MapWithMove.hs#L138 https://github.com/reflex-frp/reflex/blob/develop/src/Reflex/Patch/MapWithMove.hs#L145

simple BeagleBone Black support by copying bonegreen

2

wonderful project, thank you for it!

VSCode 1.82 and Remote 0.106.1 (or earlier?) compatibility

11

It looks like today's VSCode 1.82 release changes the location that the servers are deployed and thus breaks this. Previously, VSCode deployed servers to `~/.vscode-server/bin` but now it seems that...

Fixes #44 by not splitting the interface name before applying the modification function

`interfaceNameModifier` is documented as "Function applied to generate interface names" and so I'd expect it to apply to the whole interface name, but the call to it for `TSInterfaceDeclaration` splits...

`Instant::checked_add_duration` claims to check for overflow, doesn't

2

use of `wrapping_add` where the bases are equal, here: https://github.com/korken89/fugit/blob/master/src/instant.rs#L171 and similarly if bases are different, here: https://github.com/korken89/fugit/blob/master/src/instant.rs#L181 claim that it checks for overflow, here: https://github.com/korken89/fugit/blob/master/src/instant.rs#L156 evcxr transcript showing that...

CSV parsing doesn't support quoting of separator like RFC4180 specifies

5

E.g. the row: ``` foo,"bar,baz",qux ``` will be parsed as `["foo", "\"bar", "baz\"", "qux"]`

plus rename `text'` to `text` and move the non-overlapping instances of 'ReactText' over. so, in the end it would be: ``` haskell class ToReactNode a where node :: a ->...

Right now there's a bit of duplication and inconsistency in the naming when there's a primitive binding version of something, e.g. `ReactSpec`, `Props`, and an ease-of-use layer on top, e.g....

And this means it'll enumerate many more possible values then it should: ``` Composite.Aeson VitalPrelude ABE> data Foo2 = Foo | Bar Word8 deriving (Eq, Ord, Generic, Show) Composite.Aeson VitalPrelude...