Fix Communication JobRouter Sample Issue#34116
Conversation
|
API change check API changes are not detected in this pull request. |
|
@rohit-ganguly , @lmazuel , @lfraleigh , @mayurid , @sarkar-rajarshi Could you help review this PR and merge it if it looks good? |
|
Hi @jerryshia. Thank you for your interest in helping to improve the Azure SDK experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment. Otherwise, we'll close this out in 7 days. |
|
Hi @jerryshia. Thank you for your contribution. Since there hasn't been recent engagement, we're going to close this out. Feel free to respond with a comment containing |
|
@rohit-ganguly , @lmazuel , @lfraleigh , @mayurid , @sarkar-rajarshi Could you help review this PR and merge it if it looks good? |
|
@sarkar-rajarshi Could you help merge this PR? Thank a lot. |
rohit-ganguly
left a comment
There was a problem hiding this comment.
Looks good to me, thanks for updating.
|
@v-xuto The ci is failing with lint errors |
|
@sarkar-rajarshi https://dev.azure.com/azure-sdk/public/_build/results?buildId=3841008&view=results The same error occurs in main branch,this lint error was not caused by our PR changes. |
|
@lmazuel Could we get this PR merged in, while we address the lint errors in a separate PR? This PR simply updates some variable names in the samples? |
|
Force merging, as the changes in this PR are ok |
* fix issue33658 * fix issue33658 2
Fixes issue #33658
@rohit-ganguly , @lmazuel , @lfraleigh , @mayurid , @sarkar-rajarshi for notification.