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

Remove unused use-throttle dep from yarn.lock #719

Merged
merged 1 commit into from
Jan 25, 2021

Conversation

indiesquidge
Copy link
Collaborator

A fresh clone and install of this library results in a yarn.lock diff
with the use-throttle dep removed. This makes sense, as it's not used
anywhere in the source code nor is it a dep of any other dep.

The strange part is why this diff wasn't showing up sooner given that a
blame shows the most recent change to this part of the lock file
was 7 months ago.

A fresh clone and install of this library results in a yarn.lock diff
with the `use-throttle` dep removed. This makes sense, as it's not used
anywhere in the source code nor is it a dep of any other dep.

The strange part is why this diff wasn't showing up sooner given that a
[blame][1] shows the most recent change to this part of the lock file
was 7 months ago.

[1]: https://github.com/reach/reach-ui/blame/881b6ee3496c4b5b868f5ca1f706af63ea48114e/yarn.lock#L16431
@codesandbox-ci
Copy link

codesandbox-ci bot commented Jan 4, 2021

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit b92d794:

Sandbox Source
reach-ui-template Configuration

@chaance chaance added the Type: Housekeeping General housekeeping label Jan 25, 2021
@chaance chaance merged commit 05f5d95 into reach:develop Jan 25, 2021
@indiesquidge indiesquidge deleted the sync-yarn-lock branch February 16, 2021 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Housekeeping General housekeeping
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants