Misleading Doc: On sidecars, the depends_on attributes is not on image attribute

[alquerci]

Actual

On the doc
https://aws.github.io/copilot-cli/docs/developing/sidecars/#image-depends-on

image:
  build: ./Dockerfile
  depends_on:
    nginx: start
    startup: success

Expected/Working

sidecars:
  some_sidecar:
    image:
      build: ./Dockerfile
    depends_on:
      nginx: start
      startup: success

[Lou1415926]

hi @alquerci ! Good eyes - thank you very much for reporting this! While we pick this up, feel free to contribute a PR too! Contributions are always welcomed and very appreciated. It should be removed from

copilot-cli/site/content/docs/include/image-config.en.md

Line 44 in b1c4c42

<span class="parent-field">image.</span><a id="image-depends-on" href="#image-depends-on" class="field">`depends_on`</a> <span class="type">Map</span>

and added to https://github.com/aws/copilot-cli/blob/mainline/site/content/docs/include/sidecar-config.en.md.

Thanks again for spotting this!

[alquerci]

@Lou1415926 well it is not so simple.

The template image is good. But it is included inside sidecars template. Only on sidecars depends_on is not present inside image.

It was a good idea to avoid duplication.

So a way to keep avoiding duplication is creating an abstract common image template with same fields used on both concrete image Map.

I accept your invitation to contribute.