X Tutup
Skip to content

[release/v7.4]Add a parameter that skips verify packages step#24803

Merged
TravisEz13 merged 2 commits intorelease/v7.4from
backport-24763-74
Jan 16, 2025
Merged

[release/v7.4]Add a parameter that skips verify packages step#24803
TravisEz13 merged 2 commits intorelease/v7.4from
backport-24763-74

Conversation

@TravisEz13
Copy link
Member

Backport #24763

This pull request includes changes to the .pipelines/apiscan-gen-notice.yml and .pipelines/templates/compliance/generateNotice.yml files to introduce new parameters and improve the flexibility of the pipeline configuration. The most important changes include adding new parameters for debugging and skipping package verification, and modifying job conditions based on these parameters.

Pipeline configuration improvements:

  • Added FORCE_CODEQL and SkipVerifyPackages parameters to .pipelines/apiscan-gen-notice.yml for debugging and skipping package verification.
  • Modified the extends: section in .pipelines/apiscan-gen-notice.yml to pass the SkipVerifyPackages parameter to the compliance template.

Compliance template updates:

  • Added SkipVerifyPackages parameter to .pipelines/templates/compliance/generateNotice.yml.
  • Updated job conditions in .pipelines/templates/compliance/generateNotice.yml to skip package verification if SkipVerifyPackages is set to true.
  • Cleaned up comments and formatting in .pipelines/templates/compliance/generateNotice.yml.

* added a parameter that skips verify packages step

* fix parameter string to boolean value

---------

Co-authored-by: Justin Chung <chungjustin@microsoft.com>
@TravisEz13
Copy link
Member Author

/azp run PowerShell-CI-macos, PowerShell-CI-Linux, PowerShell-CI-Windows, PowerShell-CI-static-analysis, PSResourceGet ACR, PowerShell-Windows-Packaging-CI

@azure-pipelines
Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@TravisEz13 TravisEz13 added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Jan 16, 2025
@TravisEz13 TravisEz13 merged commit 8d74036 into release/v7.4 Jan 16, 2025
8 checks passed
@TravisEz13 TravisEz13 deleted the backport-24763-74 branch January 16, 2025 23:53
@microsoft-github-policy-service
Copy link
Contributor

microsoft-github-policy-service bot commented Jan 16, 2025

📣 Hey @TravisEz13, how did we do? We would love to hear your feedback with the link below! 🗣️

🔗 https://aka.ms/PSRepoFeedback

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

Labels

CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

X Tutup