Closed cormacpayne closed 1 year ago
Not directly related to this PR, but as I was reading through the task's flow I saw that this error message isn't localized. Is that intentional? https://github.com/Azure/container-apps-deploy-pipelines-task/blob/main/azurecontainerapps/src/Utility.ts#L28
Description
This PR focuses on the following changes:
az containerapp up
to create/update Container Apps and instead use theaz containerapp create/update
commands to give us more control over the properties that can be passed inyamlConfigPath
argumentingress
argumentIssue
N/A
Building the task
npm install
was ran within theazurecontainerapps
folder to regenerate thepackage-lock.json
filetsc
was ran within theazurecontainerapps
folder to ensure all TypeScript was successfully compiled into JavaScriptTesting the task
AzureContainerAppsTest
task was generated with these changes and publishedmocha
testsDeploying the task
inputs
array was updated intask.json
to reflect the new argument(s)inputs
array was updated intask.loc.json
to reflect the new argument(s)loc
properties were updated inresources.resjson
to reflect the new argument(s)version
property was updated intask.json
to reflect the new versionversion
property was updated intask.loc.json
to reflect the new versionversion
property was updated invss-extension.json
to reflect the new versionreleaseNotes
property was updated intask.json
to reflect the changes made since the last releasereleaseNotes
property was updated inresources.resjson
to reflect the changes made since the last release