search

Azure / container-apps-deploy-pipelines-task

Azure Pipelines Task (Release Candidate) for building and deploying Azure Container Apps
MIT License
6 stars 10 forks source link

add windows support for container tasks #5

Closed icornett closed 1 year ago

icornett commented 1 year ago

Added a new method to attempt to detect Windows agents and download an appropriate version of pack which is unpacked using PowerShell 7

icornett commented 1 year ago

Attempts to resolve #6

cormacpayne commented 1 year ago

@icornett Hey Ian, thanks again for making this change -- I just opened a PR with a bunch of miscellaneous changes from a variety of feedback sources, and bundled this change into it (since I had to tweak a few things): https://github.com/Azure/container-apps-deploy-pipelines-task/pull/10

The commit specific to your changes (and the corresponding issue you filed) can be found in this commit: https://github.com/Azure/container-apps-deploy-pipelines-task/pull/10/commits/c4e6839c6bb66ea4bd8fdd847b93b865b78e17cd

I was going to close this PR in favor of the one I linked above, but wanted to make sure it was OK with you and that you felt your contributions to the task weren't getting lost. Just let me know!

icornett commented 1 year ago

Sounds good to me