Skip to content

Fix for #3446 - #3600

Merged
Sreedhar Ande (sreedharande) merged 2 commits into
Azure:masterfrom
samikroy:patch-15
Feb 2, 2022
Merged

Fix for #3446#3600
Sreedhar Ande (sreedharande) merged 2 commits into
Azure:masterfrom
samikroy:patch-15

Conversation

@samikroy

@samikroy Samik Roy (samikroy) commented Dec 2, 2021

Copy link
Copy Markdown
Contributor

Fix for #3446

@v-sabiraj

v-sabiraj commented Dec 8, 2021

Copy link
Copy Markdown
Contributor

Hey Samik Roy (@samikroy), We have resolved the issue for PR Validations failure and request you to please update your PR branch with the latest changes from Azure:master so that the PR Validations can be executed.... Thanks!!!

@samikroy

Samik Roy (samikroy) commented Dec 8, 2021

Copy link
Copy Markdown
Contributor Author

Hey Samik Roy (@samikroy), We have resolved the issue for PR Validations failure and request you to please update your PR branch with the latest changes from Azure:master so that the PR Validations can be executed.... Thanks!!!

Thank you v-sabiraj , the build error is fixed now.

@samikroy

Samik Roy (samikroy) commented Dec 8, 2021

Copy link
Copy Markdown
Contributor Author

Hey Samik Roy (@samikroy), We have resolved the issue for PR Validations failure and request you to please update your PR branch with the latest changes from Azure:master so that the PR Validations can be executed.... Thanks!!!

Thank you v-sabiraj , the build error is fixed now. Could you please help approving this PR.

@samikroy

Samik Roy (samikroy) commented Dec 17, 2021

Copy link
Copy Markdown
Contributor Author

aprakash13 - Could you please help approving this PR.

@aprakash13

aprakash13 commented Dec 30, 2021

Copy link
Copy Markdown
Contributor

This being a playbook related change will let Sreedhar Ande (@sreedharande) take a look and approve/merge.

@shainw Shain (shainw) added the Playbook Playbook specialty review needed label Dec 30, 2021
@samikroy

Samik Roy (samikroy) commented Jan 4, 2022

Copy link
Copy Markdown
Contributor Author

Sreedhar Ande (@sreedharande) - Could you please help merging this. Thank you.

@samikroy

Samik Roy (samikroy) commented Jan 18, 2022

Copy link
Copy Markdown
Contributor Author

Sreedhar Ande (@sreedharande) - Could you please approve this.

@samikroy

Samik Roy (samikroy) commented Jan 26, 2022

Copy link
Copy Markdown
Contributor Author

aprakash13 , Shain (@shainw) , Sreedhar Ande (@sreedharande) - Request your help to approve & merge this.

@sreedharande
Sreedhar Ande (sreedharande) merged commit 592d819 into Azure:master Feb 2, 2022
@samikroy

Samik Roy (samikroy) commented Feb 2, 2022

Copy link
Copy Markdown
Contributor Author

Thank you for the approval & merge Sreedhar Ande (@sreedharande)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dicolanl SecurityJedi (dicolanl) Awaiting requested review from dicolanl
@sarah-yo Sarah Young (sarah-yo) Awaiting requested review from sarah-yo
@Yaniv-Shasha Yaniv Shasha (Yaniv-Shasha) Awaiting requested review from Yaniv-Shasha
1 more reviewer
@sreedharande Sreedhar Ande (sreedharande) sreedharande approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@sreedharande Sreedhar Ande (sreedharande)

Labels

Playbook Playbook specialty review needed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@samikroy @v-sabiraj @aprakash13 @sreedharande @shainw