await is more readable than the explicit use of promises. Also, this works towards flipping strictNullChecks to true, because TypeScript complains a bit less about the promisified execFile than about explicit uses of callbacks and promises.
While at it, this commit also moves the getBazelInfo function from debugger/client.ts over to the BazelInfo class, where it can be more easily reused. This has the nice side-effect, that also this info call will now respect any startupOptions potentially configured by the user.
awaitis more readable than the explicit use of promises. Also, this works towards flippingstrictNullChecksto true, because TypeScript complains a bit less about the promisifiedexecFilethan about explicit uses of callbacks and promises.While at it, this commit also moves the
getBazelInfofunction fromdebugger/client.tsover to theBazelInfoclass, where it can be more easily reused. This has the nice side-effect, that also thisinfocall will now respect anystartupOptionspotentially configured by the user.