flatbuffers
flatbuffers copied to clipboard
[TS/Rust/Go/Dart] Fix handling of +/-inf defaults in codegen
+/-inf were not being handled, and so invalid typescript was being generated when a float/double had an infinite default value. NaN was being handled correctly.
And I see Rust, Go, and Dart also don't handle this correctly.....
Okay, everything is passing except for the Go test, and I must be blind because I cannot figure out what the Go test is failing on from looking at the output.
Okay, everything is passing except for the Go test, and I must be blind because I cannot figure out what the Go test is failing on from looking at the output.
They seem to be the same up to addresses
(I copied the output, changed spaces to newlines, and used diff
to compare)
The Kotlin MacOS build looks just flaky, not sure how to rerun it (other than just making a dummy commit or rebasing on mater again or some similarly silly thing).
@CasperN @dbaileychess The Kotlin Linux build seems to have flaked out this time; not quite sure what the appropriate response from me is when builds fail in this manner, or if I've just gotten particularly unlucky with flakiness to have two different Kotlin tests flake out in a row.
I updated that runner https://github.com/google/flatbuffers/commit/e54536127ccb66360146d1d0fc486f126d3f4c41 due to this thread: https://github.com/gradle/wrapper-validation-action/issues/40
Sorry, i mistyped the fix, rerunning the CI.