Tim Fish

Results 417 comments of Tim Fish

I did try that but it errors. ``` // snip module.exports = { commands: { clean: run('del-cli ./dist/**', 'another command'), 'build-ts': run('tsc --pretty') } }; ``` and then: ```shell >...

@xaviergonz Looks good, I'll give it a try! Currently in my npm scripts I have `build:ts`, `build:html`, `build:scss` plus more and then `build` which calls all those in parallel. Underscores...

Thanks! Just realised for wildcard searching I have to be careful not to call the current function and create a huge recursive crash!

In Electron, the page title can automatically be used for the window title and the current ordering makes less sense in this case. I'm guessing the existing order favours the...

I don't think any of the existing PRs will fix this. My best guess is that esbuild closely follows the spec whereas other bundlers paper over the mess. The obvious...

> * Auto remove source maps after upload? I think this would be key. If you force enable source maps, they should not end up in the output. I've seen...

It's worth noting that you can disable the ESM module hooks by setting `_sentryEsmLoaderHookRegistered` before calling `Sentry.init` ```ts globalThis._sentryEsmLoaderHookRegistered = true; Sentry.init({ ``` You will lose OpenTelemetry auto-instrumentation but you'll...

There are outstanding PRs for `import-in-the-middle` that hopefully fix a wide range of issues. There is a patch available [here](https://github.com/getsentry/sentry-javascript/issues/12242#issuecomment-2133993135) that can be used with `patch-package` that should fix all...

@danilofuchs can you give more details of your setup so I can try and reproduce this? **EDIT** No worries, I've reproduced it using Node v22.2.0

Sorry, I found a merge bug in that patch. Could you try it again?