Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Use Updated npm Trim #73

Merged
merged 2 commits into from
Jan 14, 2021
Merged

Use Updated npm Trim #73

merged 2 commits into from
Jan 14, 2021

Conversation

davidcui1225
Copy link
Contributor

Issue #, if available:
N/A
Description of changes:

  • Add updated npm trim package to package.json and force it used in resolutions to make sure it is used upstream
  • Forked repos cannot be added as dependencies in yarn.lock, removed forked repo dependency in package.json

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@davidcui1225 davidcui1225 merged commit b3dc6b7 into opendistro-for-elasticsearch:dev Jan 14, 2021
@davidcui1225 davidcui1225 deleted the add-trim-resolution branch January 14, 2021 19:08
joshuali925 pushed a commit that referenced this pull request Feb 1, 2021
@davidcui1225 @joshuali925
Use Updated npm Trim (#73)
7463d85
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@joshuali925 joshuali925 joshuali925 approved these changes

@chloe-zh chloe-zh Awaiting requested review from chloe-zh

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@davidcui1225 @joshuali925