Skip to content

Create compliance build - #16286

Merged
Travis Plunk (TravisEz13) merged 18 commits into
PowerShell:masterfrom
TravisEz13:create-compliance-build
Oct 28, 2021
Merged

Create compliance build#16286
Travis Plunk (TravisEz13) merged 18 commits into
PowerShell:masterfrom
TravisEz13:create-compliance-build

Conversation

@TravisEz13

@TravisEz13 Travis Plunk (TravisEz13) commented Oct 21, 2021

Copy link
Copy Markdown
Member

PR Summary

Create daily compliance build which:

  1. Creates a notice file and puts it in a blob (will use a GitHub workflow to update in master)
  2. moves policheck and credscan to this build

PR Context

PR Checklist

@daxian-dbw Dongbo Wang (daxian-dbw) added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Oct 22, 2021

@rjmholt Rob Holt (rjmholt) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actual third party additional attributions code looks sound, just some comments on style and documentation (the latter I think is going to be important to maintain this)

Comment thread assets/additionalAttributions.txt Outdated
Comment thread tools/ComponentGovernance/ComponentGovernance.psm1 Outdated
Comment thread tools/ComponentGovernance/ComponentGovernance.psm1 Outdated
Comment thread tools/ComponentGovernance/ComponentGovernance.psm1 Outdated
Comment thread tools/releaseBuild/azureDevOps/compliance.yml Outdated
Comment thread tools/releaseBuild/azureDevOps/templates/compliance/compliance.yml Outdated
Comment thread tools/releaseBuild/azureDevOps/templates/compliance/generateNotice.yml Outdated
Comment thread tools/releaseBuild/azureDevOps/templates/compliance/generateNotice.yml Outdated
@TravisEz13 Travis Plunk (TravisEz13) added MustHave Compliance Related to compliance requirements labels Oct 27, 2021

@daxian-dbw Dongbo Wang (daxian-dbw) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Please resolve the conflicts.

@TravisEz13
Travis Plunk (TravisEz13) merged commit 5ce5936 into PowerShell:master Oct 28, 2021
@ghost

Copy link
Copy Markdown

🎉v7.2.1 has been released which incorporates this pull request.:tada:

Handy links:

@ghost

Copy link
Copy Markdown

🎉v7.3.0-preview.1 has been released which incorporates this pull request.:tada:

Handy links:

TrapGodBrim pushed a commit to TrapGodBrim/PowerShell that referenced this pull request Jan 19, 2022
Co-authored-by: Robert Holt <rjmholt@gmail.com>
Dongbo Wang (daxian-dbw) pushed a commit to daxian-dbw/PowerShell that referenced this pull request Mar 9, 2022
Co-authored-by: Robert Holt <rjmholt@gmail.com>
Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj/PowerShell that referenced this pull request Aug 6, 2026
Co-authored-by: Robert Holt <rjmholt@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backport-7.2.x-Done CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log Compliance Related to compliance requirements MustHave

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants