James Irwin

Results 9 comments of James Irwin

This has been super-ceeded by a full redesign of the upgrade modal

@rbrishabh What @RiZKiT pointed to does seem to work for this. You can put the following code in your astro.config.mjs: ``` import { defineConfig } from "astro/config"; export default defineConfig({...

@Fox32 Pinging you since I think you introduced it in your [PR](https://github.com/open-telemetry/opentelemetry-js-contrib/pull/1728) a week ago. Pinging @david-luna as well as you may have context here having approved it. Thanks for...

@david-luna Yes. Sorry. I was trying at one point to make a simpler app to reproduce it but the `api_1.context.active()` was not being set for it. So I failed. But...

For what it's worth here is code within Mongoose that handles this for the spread operator on docs: https://github.com/Automattic/mongoose/blob/e4462c6c80b8b395187b382f7dcb3c398293cd78/lib/helpers/document/handleSpreadDoc.js#L5 ScrubStatement is not using the spread operator but perhaps similar code...

We have suffered from the circular import issue before and @jdorn was able to find the culprit. I'm pretty sure the errors are coming from me adding the datasource.d.ts file...

I fixed the circular import issues. While these defaults are what we have for the front-end, I found out talking with @lukesonnet that a lot of users get into trouble...

Hrmm it seems to have crashed more widely. There were some errors in the logs, after restart. Now it has started working again. The errors were: ``` 2025-12-09 12:28:15.962 [error]...

hrmm, the downside is that this took two minutes when I was trying to check in 109 files....