Sync eng/common directory with azure-sdk-tools for PR 1031

[azure-sdk]

Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#1031
See eng/common workflow

[jsquire]

Closing; superseded by #15284

[ghost]

Hello @jsquire!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

[jsquire]

/check-enforcer override

[weshaggard]

@jsquire thanks for reviewing and merging but these sync PR's are supposed to follow the new workflow defined at https://github.com/Azure/azure-sdk-tools/blob/master/eng/common/README.md#workflow and the tools PR owner should be the one merging them. Merging them earlier causes some issues with our syncing and also doesn't allow the tools owner to finish their testing before merging the common changes.

[jsquire]

Apologies for overstepping. My understanding was that testing and review were done on merging into the central repo and that these syncs were just mechanical. I'll avoid taking direct action going forward.

Can we change to the new Central-EngSys tag and assign the PR owner when this is generated? We're trying to improve on the number of PRs that stall in the repository, many of which seem to be the automated increment or these sync items. Having a clear line of ownership would be helpful in identifying the correct person to ping for information.

[weshaggard]

Thanks for the feedback @jsquire we are definitely planning to do that. I know @chidozieononiwu is working on adding PR owners support so that we can use it for these automated PRs. @chidozieononiwu we should also add the new Central-EngSys label to these PRs.