Skip to content

Conversation

@TravisEz13
Copy link
Member

@TravisEz13 TravisEz13 commented Oct 26, 2020

PR Summary

Enable CodeQL Security Scanning

PR Context

PR Checklist

@ghost ghost assigned anmenaga Oct 26, 2020
@TravisEz13 TravisEz13 marked this pull request as ready for review October 26, 2020 19:04
@TravisEz13 TravisEz13 requested a review from PaulHigin October 26, 2020 20:36
Copy link
Contributor

@PaulHigin PaulHigin left a comment

Choose a reason for hiding this comment

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

LGTM

name: Capture Environment

- run: |
[Net.ServicePointManager]::SecurityProtocol = [Net.ServicePointManager]::SecurityProtocol -bor [Net.SecurityProtocolType]::Tls12
Copy link
Member

Choose a reason for hiding this comment

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

Please remove

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed here and in the CI build template

@TravisEz13 TravisEz13 merged commit 191c00d into master Oct 27, 2020
@TravisEz13 TravisEz13 deleted the TravisEz13-patch-2 branch October 27, 2020 17:20
@iSazonov iSazonov added the CL-Tools Indicates that a PR should be marked as a tools change in the Change Log label Oct 29, 2020
@iSazonov iSazonov added this to the 7.2.0-preview.1 milestone Oct 29, 2020
@ghost
Copy link

ghost commented Nov 17, 2020

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

Handy links:

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

Labels

CL-Tools Indicates that a PR should be marked as a tools change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants