[TESTING ONLY] Testing common changes required for duplicate artifactId

[JimSuplizio]

I'm testing changes to API review creation. These changes will pass a list of PackageInfo files instead of a list of Artifact.Names. This allows passing around the PackageInfo object instead random pieces of data. Passing around the PackageInfo object also allows the FindArtifactForApiReviewFnV2 to correctly, and easily, find the sources jar since the PackageInfo file contains the Group. This eliminates the janky code that had to exist in the original FindArtifactForApiReviewFn which had to play a guessing game with the group to get correct path to the sources.jar

The other changes are to update the detect-api-changes, updating to use the FindArtifactForApiReviewFnV2 instead of the FindArtifactForApiReviewFn also, in some cases, pass the PackageInfo file to functions so we're not loading the same PackageInfo multiple times.

validate-all-packages changes

• Validate-All-PackagesV2.ps1 takes in a list of PackageInfo files.
• Validate-All-PackagesV2.ps1 folds in what was originally in Validate-Package.ps1, which is only called by Validate-All-Packages, into the script instead of making another script call.
• Refactor what was in Update-DevOps-Release-WorkItem.ps1, moving the contents of this script into a function, Update-DevOpsReleaseWorkItem, in DevOps-WorkItem-Helpers.ps1
• Move the az login and azure-devops extension checks into a function, CheckAzLoginAndDevOpsExtensionInstall, in DevOps-WorkItem-Helpers.ps1.
• CheckAzLoginAndDevOpsExtensionInstall and CheckDevOpsAccess both check, and set, global variables to see if their respective actions have already been done.
• Part of the refactoring work done here is because both Validate-Package.ps1 and Update-DevOps-Release-WorkItem.ps1, when called as separate scripts, both had to az login, azure-devops extension install and devops access. Because these were separate scripts calls, there's no real way to save any kind of state so all of these checks were made for every artifact being processed.

Testing changes to this, at least for create-apireview.yml, requires a pipeline to be manually kicked off against the refs/merge of this PR to test yml path for PackageInfo files and the new script.

[weshaggard]

Let's take and convert this ArtifactList to a set of PackageInfoFiles and eliminate the need to bifurcate the scripts.

[weshaggard]

You are already doing the mapping from artifact to package info right here. You can do that for everyone and not need to bifurcate all the scripts.

[JimSuplizio]

I love how well rebase works. Closing this PR.