Vasek - Tom C
Vasek - Tom C
I added a commit with theses changes, it works locally, I'm currently checking if it passes on the CI
Only 1 test failing, I can reproduce locally with dagger call but not with a manual test, this is weird https://dagger.cloud/dagger/traces/9144e633456d21a1f0ab7ce11bb0a7cd?span=a29638ceb3992471#a49f16c55743f48b ``` dagger call version Full trace at https://dagger.cloud/ (rotate...
Ok I normally fixed the tests, it seems it was related to the typescript version installed with that specific bun version, I fixed it by adding the deps to the...
> This would be different than the existing [ContextDirectory method](https://github.com/sipsma/dagger/blob/5231aa7de19266430581da054604ce533cc85898/core/modulesource.go#L247-L247) that returns the static one loaded for the SDKs (we can consider renaming it maybe for clarity). I understand this...
> Relative paths are relative to the current source file, scoped to the context directory for security To do such thing, we need to know the file where the module...
> Sorry I forgot to update that line after further discussion. It should be relative to the module root, not the file (that would be too difficult to find out)....
Another question @helderco @jedevc @shykes, should the API returns an error during registration if the typedef has both a default value and a contextual one? This is possible in TypeScript...
> I thought we were using default values? I might have missed (or forgotten) the last round of bikeshedding on that topic. No it's a specific field in the API,...
> @TomChv could you add example snippets of what that feature would look like, in the 3 main languages please? It would make it easier to discuss the DX tradeoffs...
> @TomChv could you add an example of how regular default values work in each case? Example on https://docs.dagger.io/manuals/developer/functions#default-values **Go** I'm not sure you can default to a directory value...