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

Upgrade mem to version 4.0.0. #5658

Closed
wants to merge 1 commit into from
Closed

Upgrade mem to version 4.0.0. #5658

wants to merge 1 commit into from

Conversation

kittaakos
Copy link
Contributor

See: sindresorhus/memoize/pull/19.
Signed-off-by: Akos Kitta kittaakos@typefox.io

See: sindresorhus/memoize/pull/19.
Signed-off-by: Akos Kitta <kittaakos@typefox.io>
@akosyakov
Copy link
Member

akosyakov commented Jul 8, 2019

I don't think that fixes vulnerability for downstream projects, only for this repo. I think we should upgrade a dependency which leaks vulnerability to fix it in this repo and for dependent projects.

@akosyakov akosyakov added the security issues related to security label Jul 8, 2019
@kittaakos
Copy link
Contributor Author

I don't think that fixes vulnerability for downstream projects, only for this repo. I think we should upgrade a dependency which leaks vulnerability to fix it in this repo and for dependent projects.

OK, make sense. Let's do the correct way; I hope someone has a better solution than this quick-fix.

@kittaakos kittaakos closed this Jul 8, 2019
@kittaakos kittaakos deleted the update-mem branch July 8, 2019 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
security issues related to security
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants