Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update pyyaml dev requirement to fix GitHub security warning #18

Merged
merged 2 commits into from
Feb 14, 2020

Conversation

bgrant
Copy link
Collaborator

@bgrant bgrant commented Feb 13, 2020

Resolves GitHub security warning.

Remediation
Upgrade pyyaml to version 4.2b1 or later. For example:

pyyaml>=4.2b1

This is a dev dependency, so I don't see why we shouldn't upgrade to the latest.

Resolves GitHub security warning.
@salesforce-cla
Copy link

Thanks for the contribution! It looks like @bgrant is an internal user so signing the CLA is not required. However, we need to confirm this.

@bgrant bgrant requested a review from demianbrecht February 13, 2020 21:00
@bgrant bgrant changed the title Update pyyaml dev requirement Update pyyaml dev requirement to fix GitHub security warning Feb 13, 2020
@bgrant bgrant merged commit b6d9c3f into master Feb 14, 2020
@bgrant bgrant deleted the feature/update-pyyaml branch February 14, 2020 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants