Vasek - Tom C

Results 236 comments of Vasek - Tom C

Thanks for this amazing issue @gerhard I think https://github.com/dagger/dagger/pull/8185 should also be include to the effort and then spread globally to other test suite. Moving TypeScript related test from `TestModule`...

What is the meaning of the `13%` in the table? This [comment](https://github.com/dagger/dagger/issues/8184#issuecomment-2324794092) seems to show that it's actually doing better that ever?

@zoupingshi Hey 👋 I'm happy to merge that PR once you finished to rebase it :)

I'm closing this PR since I noticed there's no activity since a month. I tried to rebase it myself but conflicts are so big that it would be simpler to...

> What issue are you running into with yarn v4? A potential issue that you will probably run into is that since the users module has a @dagger.io/dagger in dependencies,...

> One of the issues is due to the fact that that you aren't using the node_modules linker. To enable that you would need a .yarnrc file with Okay, I'll...

Update: on test failing left: `TestTypescriptWithOtherModuleTypes` My guess is that by resolving the modules differently, it fails to provide the corresponding symbol of the dependencies, I'm trying to find a...

Ok the bug should be fixed! There was an issue with the alias resolution.

> It's not really a yarn-v4 thing. When install (for any package manager) is run, it will install the dependencies for the sdk as well since its listed as a...

> So pnpm isolates its dependencies, so that may not be an issue, but I'm pretty sure that npm does. It may still work, but its not going to pick...