Mike Bender

Results 61 comments of Mike Bender

This won't be moving along until https://github.com/deephaven/deephaven-core/issues/3487 is completed, removing from April milestone.

We do have a rudimentary pivot table builder created using deephaven.ui: https://gist.github.com/dgodinez-dh/9765c4a3de19b224d1e9bd77efca65b7

@niloc132 doesn't look like we read minRange or maxRange anywhere

We cannot use a dist-tag of `v0.33` for example, otherwise we get an error when publishing: ``` Run npm publish --tag v0.33 web/client-api/types/build/deephaven-jsapi-types-*.tgz npm ERR! Tag name must not be...

Fix would just be changing `latest` to the release branch name at https://github.com/deephaven/deephaven-core/blob/bf13489d48327e2e3d504e6be13020f30c46bd23/.github/workflows/publish-ci.yml#L120 (e.g. `v0.33`)

From @devinrsmith : > May need to be a bash script. You'll want to get the official version from gradle (as opposed to the branch name)

Tests are already written as actions and we have docker wiring setup in both web-client-ui and deephaven-plugins repos: Web-client-ui: https://github.com/deephaven/web-client-ui/blob/main/.github%2Fworkflows%2Fe2e.yml Deephaven-plugins: https://github.com/deephaven/deephaven-plugins/blob/main/.github%2Fworkflows%2Fe2e.yml It only runs against edge though, so we...

The jsapi-types version is intentionally in sync with the deephaven-core version. We would lose that with a nightly release. We simply should not have declared the `as any` on the...

@AkshatJawne Run `npm run e2e:update-ci-snapshots` to update the snapshots: https://github.com/deephaven/web-client-ui/blob/main/README.md#e2e-tests