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

Configure Snyk protect to enable patches #33

Merged
merged 2 commits into from
Feb 15, 2019

Conversation

snyk-bot
Copy link
Contributor

We noticed that you have patches available for this project, however it seems that snyk protect is not properly configured.

We're adding snyk protect in order to patch your code and make it more secure.

If you intentionally removed snyk protect, then you can close this PR instead.

If you are using a lockfile please pull down this branch and update the lockfile by running npm i or yarn install.

PS. We'd love to find out what you think about snyk protect, especially if it hasn't proved useful. Please get in touch via support@snyk.io

@BrandonSharratt BrandonSharratt added the question Further information is requested label Jan 31, 2019
@jujaga jujaga requested a review from ikethecoder February 14, 2019 08:06
@jujaga jujaga added the enhancement New feature or request label Feb 14, 2019
@jujaga jujaga changed the base branch from master to develop February 14, 2019 08:35
@jujaga jujaga force-pushed the fix/snyk-protect-enable-patches-mz12u8 branch from 1016fc7 to 6d569e2 Compare February 14, 2019 22:50
Copy link
Member

@jujaga jujaga left a comment

Choose a reason for hiding this comment

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

Both @Jonesy and I have evaluated this change and approve of it.

@jujaga jujaga merged commit e514107 into develop Feb 15, 2019
@jujaga jujaga deleted the fix/snyk-protect-enable-patches-mz12u8 branch February 15, 2019 02:32
@jujaga jujaga added this to the Sprint 49 milestone Feb 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants