Quickstart tutorial: Problem with Simplify the pipeline example

[drowsy eagle]

Hi! I'm new to Dagger and am learning.
I was going through the Quickstart and ran into a problem in one of the tutorials, where the example here: https://docs.dagger.io/quickstart/simplify#simplify-the-pipeline does not compile.

I looked at the code, and saw that the problem I encountered was actually fixed here: https://github.com/dagger/dagger/commit/6ca03a8a6a81b28e900b4a5fc59c6d16442f8c3b

However, this fix isn't reflected online: https://docs.dagger.io/quickstart/simplify#simplify-the-pipeline

It's not a big deal for me, since I was able to make the same change in my local workspace and complete the tutorial. When will the change in the code be reflected online in the tutorial?

Thanks.
The tutorial is really nice, BTW. 😄

[unborn field]

Hey! Thx for reporting. Seems like we forgot to publish our latest docs since I can see the change appears on https://devel.docs.dagger.io cc @acoustic obsidian

We're also happy that you're enjoying the quick start. Let us know in case you have further questions so we can assist 🙌

[drowsy eagle]

Thanks for the quick response. Thanks again for the nice Quickstart tutorial. 😄

[acoustic obsidian]

I've republished the docs site so this should be fixed now. Thanks for letting me know @drowsy eagle and @unborn field !

[unborn field]

thx Vikram!

[drowsy eagle]

@acoustic obsidian @unborn field Thanks for the quick response and for pushing out the docs.

I'm still seeing the same problem though. At
https://docs.dagger.io/quickstart/simplify/#simplify-the-pipeline

there is this:

// Build a ready-to-use development environment
func (m *HelloDagger) BuildEnv(source *dagger.Directory) *dagger.Container {
    return dag.Node(NodeOpts{Version: "21"}).
        WithNpm().
        WithSource(source).
        Install(nil).
        Container()
}

which should be:

// Build a ready-to-use development environment
func (m *HelloDagger) BuildEnv(source *dagger.Directory) *dagger.Container {
    return dag.Node(dagger.NodeOpts{Version: "21"}).
        WithNpm().
        WithSource(source).
        Install().
        Container()
}

I thought that this problem was fixed in https://github.com/dagger/dagger/commit/6ca03a8a6a81b28e900b4a5fc59c6d16442f8c3b

Is there some other fix that is required to have this reflected in the docs?
Thanks.

[devout kettle]

i see build & test functions using Commands().Run() from sdk/typescript/dev/node/src/commands.ts;

  @func()
  run(args: string[]): Container {
    return this.ctr.withExec(["run", ...args])
  }

for ex, if we want to use npm run,
the return is exec run ...args not npm run ...args

[acoustic obsidian]

Checking this, will update here shortly @drowsy eagle

[acoustic obsidian]

Looks like it was a cache problem. I've cleared the cache and redeployed, it seems to be correct now. Can you please confirm @drowsy eagle - thanks!

also a minor fix in https://github.com/dagger/dagger/pull/8404 for you to review @unborn field