Skip to content

Conversation

@lbogdan
Copy link
Contributor

@lbogdan lbogdan commented Jan 13, 2021

No description provided.

@lbogdan
Copy link
Contributor Author

lbogdan commented Jan 13, 2021

Build for latest commit a839611 is at https://pr5359.build.csb.dev/s/new.

.concat(...stats.compilation.children.map(child => child.warnings))
.filter(warning => warning.error.name !== 'CriticalDependencyWarning');
.filter(warning => warning.error.name !== 'CriticalDependencyWarning')
// ignore "Error: Can't resolve 'sugarss' in '/client/node_modules/postcss-import/lib'" warning
Copy link
Contributor

Choose a reason for hiding this comment

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

bless you

@SaraVieira
Copy link
Contributor

It looks good! Did you test running it in the docker container to make sure it still works local?

Also maybe a good idea to revert the yarn.lock

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jan 13, 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 a839611:

Sandbox Source
Notifications Test Configuration

@lbogdan lbogdan merged commit 9420c4f into master Jan 20, 2021
@lbogdan lbogdan deleted the feat/remote-dev branch January 20, 2021 13:47
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.

4 participants