Future / proposed spec changes

[bamurtaugh]

Updating based on devcontainers/cli#228 and devcontainers/spec#89.

As discussed, I believe we want to wait to merge these changes for another few weeks, but I wanted to open the PR for discussion now.

[Chuxel]

Yeah this makes sense in these spots. 👍

For parallel scripts though, this also means that the json_reference needs a tweak since type for the "Lifecycle scripts" commands in the is now string, array, or object. There's a description of string vs array there, so I think we'd need to summarize parallel execution there too.

Like in the other case, probably makes sense to add this to the respective spots in devcontainers/spec so we don't have a snafu on copying across later. 😄

Few other comments below!

[bamurtaugh]

Thanks for the review @Chuxel! I've done the following:

• Updated lifecycle scripts info to reference object type, added info on object type to the existing section on array vs string
• Pulled in changes from Support for adding unique ID into volume mount names spec#62
• Open Latest spec changes spec#128 for associated spec changes

[Chuxel]

A few thoughts (which would also apply to devcontainers/spec#128)

[bamurtaugh]

Thanks so much for the great review and collab @Chuxel. I'll merge this PR and devcontainers/spec#128 after the next Dev Containers release next week.

[chrmarti]

LGTM, left one comment on the additional devcontainer.json properties.

[bamurtaugh]

As the release is out, merging now. Thanks!