uselagoon / build-deploy-tool

Tool to generate build resources
2 stars 5 forks source link

fix: use yq for checking container-registries because shyaml is not great #322

Closed shreddedbacon closed 1 month ago

shreddedbacon commented 1 month ago

shyaml is not great and we should stop using it. As more and more work goes into #289 the usage of shyaml is reduced significantly though.

This changes the checks to use yq where possible in the container-registry functions to handle things shyaml isn't great with.

The version of yq used is v4.35.2 which is what is used in the build-deploy-image currently

With updated gist

310 introduced the changed logic, but the version of shyaml differs across operating systems and some handled it differently.