Skip to content

Updated Signtool-exe.md to include coming crypto changes to the tool#21048

Merged
tdykstra merged 15 commits into
dotnet:masterfrom
jgeurten:master
Nov 2, 2020
Merged

Updated Signtool-exe.md to include coming crypto changes to the tool#21048
tdykstra merged 15 commits into
dotnet:masterfrom
jgeurten:master

Conversation

@jgeurten

Copy link
Copy Markdown
Contributor

Summary

*Announcing the updates coming to SignTool in the 20236 kit builds
*Aligned documentation between win32 and seccrypto SignTool exe docs
*Added examples for SignTool

Fixes #Issue_Number (if available)

I am a MSFT PM on the Windows code signing feature team (we own Signtool.exe)

@dnfadmin

dnfadmin commented Oct 12, 2020

Copy link
Copy Markdown

CLA assistant check
All CLA requirements met.

@Youssef1313 Youssef1313 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.

Fix linter errors

Comment thread docs/framework/tools/signtool-exe.md Outdated
Comment thread docs/framework/tools/signtool-exe.md Outdated
Comment thread docs/framework/tools/signtool-exe.md Outdated
Comment thread docs/framework/tools/signtool-exe.md Outdated
Comment thread docs/framework/tools/signtool-exe.md Outdated
Comment thread docs/framework/tools/signtool-exe.md Outdated
jgeurten and others added 6 commits October 13, 2020 14:59
Co-authored-by: Youssef Victor <31348972+Youssef1313@users.noreply.github.com>
Co-authored-by: Youssef Victor <31348972+Youssef1313@users.noreply.github.com>
Co-authored-by: Youssef Victor <31348972+Youssef1313@users.noreply.github.com>
Co-authored-by: Youssef Victor <31348972+Youssef1313@users.noreply.github.com>
Co-authored-by: Youssef Victor <31348972+Youssef1313@users.noreply.github.com>
Co-authored-by: Youssef Victor <31348972+Youssef1313@users.noreply.github.com>
@jgeurten

Copy link
Copy Markdown
Contributor Author

@Youssef1313 @tdykstra any update on this PR? SignTool changes are live and this doc should reflect those changes. Thanks

@tdykstra tdykstra left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, left some suggestions for style consistency.

Comment thread docs/framework/tools/signtool-exe.md Outdated
Comment thread docs/framework/tools/signtool-exe.md Outdated
Comment thread docs/framework/tools/signtool-exe.md Outdated
Comment thread docs/framework/tools/signtool-exe.md Outdated
Comment thread docs/framework/tools/signtool-exe.md Outdated
jgeurten and others added 5 commits November 2, 2020 10:30
Co-authored-by: Tom Dykstra <tdykstra@microsoft.com>
Co-authored-by: Tom Dykstra <tdykstra@microsoft.com>
Co-authored-by: Tom Dykstra <tdykstra@microsoft.com>
Co-authored-by: Tom Dykstra <tdykstra@microsoft.com>
Co-authored-by: Tom Dykstra <tdykstra@microsoft.com>
@jgeurten

jgeurten commented Nov 2, 2020

Copy link
Copy Markdown
Contributor Author

LGTM, left some suggestions for style consistency.

Thank you @tdykstra for the suggested changes! I have approved them.

@tdykstra

tdykstra commented Nov 2, 2020

Copy link
Copy Markdown
Contributor

Thanks -- I'll merge as soon as the build finishes.

@tdykstra tdykstra merged commit f318e9a into dotnet:master Nov 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants