Skip to content

Update resolved dependencies #66

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

Merged
merged 1 commit into from
Feb 27, 2019

Conversation

rplopes
Copy link
Contributor

@rplopes rplopes commented Feb 27, 2019

By deleting some of the resolutions and running yarn again, we can get the latest resolved versions as accepted by the semver notation used by the projects requiring them.

This allows us to get more up to date dependencies that fix some known security issues. Those issues were not problematic, since they were all from dev dependencies (required by babel and jest), yet it's nice to keep the project clean from them.

As a nice side effect, this ends up consolidating some of the resolved dependencies, resulting in a smaller yarn.lock.

Unverified

This user has not yet uploaded their public signing key.
@rplopes rplopes self-assigned this Feb 27, 2019
@rplopes rplopes requested a review from Americas February 27, 2019 15:04
Copy link
Contributor

@Americas Americas left a comment

Choose a reason for hiding this comment

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

😆

@Americas Americas merged commit 072be4d into master Feb 27, 2019
@Americas Americas deleted the support/update-resolved-dependencies branch February 27, 2019 15:15
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.

None yet

2 participants