◐ Shell
clean mode source ↗

move to new signing process for release build and prep for 1.20.0 by JamesWTruher · Pull Request #1625 · PowerShell/PSScriptAnalyzer

@JamesWTruher

PR Summary

The infrastructure at MS has changed with regard to signing, this PR allows us to use it.

PR Checklist

bergmeister

Choose a reason for hiding this comment

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

looks good but the build seems to be broken. Also wondering whether 1.20.0 would be a better target version since the changes are all small improvements but not fixes

Add binskim scan directory
Also publish nupkg directly from signed directory
and don't publish the out directory, publishing the signed directory is enough
Also try to publish just the nupkg

@JamesWTruher

I've rebased to be up to date.

@bergmeister, @rjmholt, @SydneyhSmith - Chris is suggesting that our next release should perhaps be number 1.20.0 and I wanted everyone to have a chance to provide an opinion. FWIW, I'm ok with 1.20

rjmholt

Choose a reason for hiding this comment

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

LGTM -- agree that 1.20 seems good

TravisEz13

Choose a reason for hiding this comment

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

looks about right to me

@bergmeister bergmeister changed the title move to new signing process for release build and prep for 1.19.2 move to new signing process for release build and prep for 1.20.0

Feb 10, 2021

bergmeister

rjmholt

Choose a reason for hiding this comment

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

Generally looks good, just a small comment about the Write-Host invocations

Co-authored-by: Robert Holt <rjmholt_msft@outlook.com>

rjmholt