Skip to content

Conversation

@StrawnSC
Copy link
Contributor

@StrawnSC StrawnSC commented Mar 23, 2022

Description

Fixes #21728

Testing Guide

History Notes


This checklist is used to make sure that common guidelines for a pull request are followed.

@ghost ghost added the Auto-Assign Auto assign by bot label Mar 23, 2022
@ghost ghost requested review from wangzelin007 and yonzhan March 23, 2022 17:42
@ghost ghost assigned zhoxing-ms Mar 23, 2022
@ghost ghost added this to the Mar 2022 (2022-04-06) milestone Mar 23, 2022
@ghost ghost added App Services az appservice Web Apps az webapp labels Mar 23, 2022
@StrawnSC StrawnSC changed the title [App Service] az webapp deployment github-actions add: Fix #21741: Allow passing in runtime with colon delimiter [App Service] az webapp deployment github-actions add: Fix #21728: Allow passing in runtime with colon delimiter Mar 23, 2022
@yonzhan
Copy link
Collaborator

yonzhan commented Mar 23, 2022

App Service

@wangzelin007
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@panchagnula
Copy link
Contributor

@calvinsID can you review the PR. @StrawnSC is there a mock test we can add to ensure that we catch such issues in the future?

@StrawnSC
Copy link
Contributor Author

@panchagnula we don't have any mocked tests covering webapp deployment github-actions add. I'll go ahead and add one to cover this

Copy link
Contributor

@calvinsID calvinsID left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@zhoxing-ms zhoxing-ms changed the title [App Service] az webapp deployment github-actions add: Fix #21728: Allow passing in runtime with colon delimiter [App Service] Fix #21728: az webapp deployment github-actions add: Allow passing in runtime with colon delimiter Mar 25, 2022
@zhoxing-ms
Copy link
Contributor

@StrawnSC Could you please resolve those CI issues?

@zhoxing-ms zhoxing-ms merged commit 9bae608 into Azure:dev Mar 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

App Services az appservice Auto-Assign Auto assign by bot Web Apps az webapp

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Runtimes documentation slightly misleading

6 participants