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.

Sign up for GitHub

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

Jump to bottom

fix: align PRUNE_DEPS_THRESHOLD with internal value #1449

Closed
wants to merge 1 commit into from
Closed

fix: align PRUNE_DEPS_THRESHOLD with internal value #1449

wants to merge 1 commit into from

Conversation

orsagie
Copy link
Contributor

@orsagie orsagie commented Oct 7, 2020 •
edited
Loading

  • Ready for review
  • Follows CONTRIBUTING rules
  • Reviewed by Snyk internal team

What does this PR do?

Aligns the threshold for pruning dependencies with our internal value of 20,000

@orsagie orsagie requested review from a team as code owners October 7, 2020 10:14
@orsagie orsagie self-assigned this Oct 7, 2020
@ghost ghost requested review from anthogez and ekbsnyk October 7, 2020 10:14
@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2020 •
edited by orsagie
Loading

Expected release notes (by @orsagie)

fixes:
align PRUNE_DEPS_THRESHOLD with internal value (8bc2b6b)

  • I hereby acknowledge these release notes are 🥙 AWESOME 🥙

@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2020

Warnings
⚠️

Looks like you added a new Tap test. Consider making it a Jest test instead. See files like test/*.spec.ts for examples. Files found:

Generated by 🚫 dangerJS against 8bc2b6b

@orsagie orsagie closed this Oct 7, 2020
@orsagie orsagie deleted the fix/align-prune-threshold-with-internal branch October 7, 2020 19:49
This was referenced Jan 17, 2022
Open
Open
This was referenced Aug 30, 2022
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anthogez anthogez Awaiting requested review from anthogez

@ekbsnyk ekbsnyk Awaiting requested review from ekbsnyk

Assignees

@orsagie orsagie

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@orsagie