Gytis Čepkauskas

Results 10 comments of Gytis Čepkauskas

> This PR looks good to me. Any reason it's still sitting in a draft state? There is a problem with Certa tool not working properly while debugging on Electron...

@calebmshafer, @aruniverse Support for Electron 18 and 19 is still blocked by `AsyncLocalStorage` problem. It seems to be a V8 bug ([Issue #35043](https://github.com/electron/electron/issues/35043#issuecomment-1203673147)). My recommendation is to add support for...

Bug should be fixed starting `@itwin/core-frontend` version `4.8.0`. Please close this issue if tool is behaving as expected.

iTwin.js does not provide functionality to override internet connectivity as both `NativeHost.overrideInternetConnectivity` and `NativeApp.overrideInternetConnectivity` are internal. Additionally, properties or methods of `NativeApp` and `NativeHost` should not be called before `startup`...

`GeometryLoadOptions` probably should be just: ```ts export interface GeometryLoadOptions { wantBRepData?: boolean; } ```

Most likely unrelated, but there were changes to TypeScript config in 5.0, which affected classes derived from `Entity` class. So if you're using tsconfig from _@itwin/build-tools_ and have classes derived...

> Any objections to "previous-minor" as the name of the tag? > > Seems good to me since we already use "previous" Should we just use "previous" tag instead of...

> @GytisCepk commit history showed you generated the TOC for 5.4 first previously, can you test this branch locally (i.e add the omit from toc comment to next ver and...

> Your PR description should specify why you want to do this. Updated description.

> Make `SnapshotConnection.openFile` only available in IPC apps. This will break integration tests of at least 1 application. I don't want to break anything on minor release, I'm moving this...