Enable add containers with split

syntasso / kratix-cli

CLI-based tool to build Promises

Apache License 2.0

2 stars 2 forks source link

Enable add containers with split #13

Closed SaphMB closed 4 months ago

SaphMB commented 4 months ago

Closes #7

SaphMB commented 4 months ago

LGTM, couple comments.

I tried adding the same container twice and the CLI didn't error or let me know it was over writing the existing container. Might be worth erroring if a user does that?

Very good spot! We've now updated the command to ensure this is no longer the case