Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

Clear redux cache before yarn install #932

Merged
merged 1 commit into from
Mar 12, 2018
Merged

Clear redux cache before yarn install #932

merged 1 commit into from
Mar 12, 2018

Conversation

GoldUniform
Copy link
Contributor

@jwilander can you have a look at this change? this clears the cache for the redux package before the yarn install command.

@GoldUniform GoldUniform added the 2: Dev Review Requires review by a core commiter label Mar 12, 2018
@GoldUniform GoldUniform added this to the v4.8.0 milestone Mar 12, 2018
@GoldUniform GoldUniform self-assigned this Mar 12, 2018
jwilander
jwilander approved these changes Mar 12, 2018
View reviewed changes
Copy link
Member

@jwilander jwilander left a comment

Choose a reason for hiding this comment

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

Looks like it might help, at the very least it won't make it worse

@GoldUniform GoldUniform merged commit 8ab93bc into release-4.8 Mar 12, 2018
@GoldUniform GoldUniform deleted the clear_redux_cache branch March 12, 2018 18:02
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Mar 12, 2018
@lindalumitchell lindalumitchell added the Tests/Not Needed Does not require new release tests label Mar 14, 2018
@lieut-data lieut-data removed their request for review April 13, 2018 16:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jwilander jwilander jwilander approved these changes

Assignees

@GoldUniform GoldUniform

Labels
2: Dev Review Requires review by a core commiter Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation Tests/Not Needed Does not require new release tests
Projects
None yet
Milestone
v4.8.0
Development

Successfully merging this pull request may close these issues.
4 participants
@GoldUniform @jwilander @lindalumitchell @amyblais