1ESPT conversion cleanup about arcade HOT 5 OPEN

Understand whether we still need to generate SBOMs with our own templates or can solely rely on the template injected sbom generation. This might not be possible, as the Microbuild guidance is to keep using the manifest generator task...

if we keep our own SBOM task then we need to bump the package version since it wasn't bumped in main: #14511

from arcade.

Adding a few more items

• Clean up any conditional that mentions running a PR, running as public, or running in the public project

• Understand whether we still need to generate SBOMs with our own templates or can solely rely on the template injected sbom generation. This might not be possible, as the Microbuild guidance is to keep using the manifest generator task...

  SBOM: The Migration Tooling will remove the SBOM Manifest Builder Task. Please put this back. The SBOM tasks in the 1ES PT does not conform to the way that we package up SBOM's into manifest files in MicroBuild.

from arcade.

re the CodeQL point: parameters.PackageVersion is a lousy name anyhow. we should at least fix that if the hook is still needed

from arcade.

As a more general cleanup point:

• the public templates should only do public things
• the official templates should only do official things

from arcade.

Understand whether we still need to generate SBOMs with our own templates or can solely rely on the template injected sbom generation. This might not be possible, as the Microbuild guidance is to keep using the manifest generator task...

if we keep our own SBOM task then we need to bump the package version since it wasn't bumped in main: #14511

Especially because a certain @riarenas said we shouldn't update it in main yet😛

from arcade.