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

fix: Add Emotion dependency to fix transient dep issue #917

Merged
merged 2 commits into from
Nov 25, 2020

Conversation

anicholls
Copy link
Contributor

Summary

Something about #897 has been causing a non-deterministic transient dependency issue in some environments. For some reason, @emotion/styled-base is not installed under some circumstances, even though it is a dependency of @emotion/styled (which is used in many of our components). We were unable to track down the source of the issue, so I've temporarily added this package as a workspace dependency to get our master builds passing again.

@anicholls anicholls changed the title fix: Add transient Emotion dependency issue fix: Add Emotion dependency to fix transient dep issue Nov 24, 2020
@cypress
Copy link

cypress bot commented Nov 24, 2020



Test summary

560 0 1 0


Run details

Project canvas-kit
Status Passed
Commit 8726145 ℹ️
Started Nov 25, 2020 12:47 AM
Ended Nov 25, 2020 12:53 AM
Duration 05:35 💡
OS Linux Ubuntu Linux - 18.04
Browser Electron 80

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

Copy link
Contributor

@willklein willklein left a comment

Choose a reason for hiding this comment

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

🙃 If we must...

package.json Outdated Show resolved Hide resolved
@anicholls anicholls merged commit b823d31 into Workday:master Nov 25, 2020
@anicholls anicholls deleted the fix-transient-dep branch November 25, 2020 18:54
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.

3 participants